[flang][msvc] Avoid templated initializer list initialization of vector. NFC.
authorMichael Kruse <llvm-project@meinersbur.de>
Tue, 29 Sep 2020 16:37:35 +0000 (11:37 -0500)
committerMichael Kruse <llvm-project@meinersbur.de>
Tue, 29 Sep 2020 16:40:42 +0000 (11:40 -0500)
The Microsoft compiler emits an error when populating the vector with a single element of a templated argument using the brace syntax. The error is:
```
constant.h(102,1): error C2664: 'std::vector<Fortran::evaluate::value::Complex<...>, ...>::vector(std::initializer_list<_Ty>,const _Alloc &)': cannot convert argument 1 from 'initializer list' to 'std::initializer_list<_Ty>'
```
To work around this error, we replace the templated constructor with one for the expected type. Conversion to the element type has to be done by the caller.

This patch is part of the series to make flang compilable with MS Visual Studio <http://lists.llvm.org/pipermail/flang-dev/2020-July/000448.html>.

Reviewed By: klausler

Differential Revision: https://reviews.llvm.org/D88163

flang/include/flang/Evaluate/constant.h

index a25916f..a9f6e87 100644 (file)
@@ -97,8 +97,7 @@ public:
 
   template <typename A>
   ConstantBase(const A &x, Result res = Result{}) : result_{res}, values_{x} {}
-  template <typename A, typename = common::NoLvalue<A>>
-  ConstantBase(A &&x, Result res = Result{})
+  ConstantBase(ELEMENT &&x, Result res = Result{})
       : result_{res}, values_{std::move(x)} {}
   ConstantBase(
       std::vector<Element> &&, ConstantSubscripts &&, Result = Result{});