[all-commits] [llvm/llvm-project] 207d44: [flang][msvc] Split class declaration and constexp...

Michael Kruse via All-commits all-commits at lists.llvm.org
Sat Aug 22 13:06:04 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 207d4499dfab050b0535c7361527d3c37cf6488a
      https://github.com/llvm/llvm-project/commit/207d4499dfab050b0535c7361527d3c37cf6488a
  Author: Michael Kruse <llvm-project at meinersbur.de>
  Date:   2020-08-22 (Sat, 22 Aug 2020)

  Changed paths:
    M flang/lib/Parser/basic-parsers.h
    M flang/lib/Parser/expr-parsers.cpp
    M flang/lib/Parser/token-parsers.h

  Log Message:
  -----------
  [flang][msvc] Split class declaration and constexpr variable definition. NFC.

Msvc has trouble defining a struct/class and defining a constexpr symbol in the same declarator. It reports the following error:
```
basic-parsers.h(809): error C2131: expression did not evaluate to a constant
basic-parsers.h(809): note: failure was caused by call of undefined function or one not declared 'constexpr'
basic-parsers.h(809): note: see usage of 'Fortran::parser::OkParser::OkParser'
```

Fix the msvc compilation by splitting the two definitions into two separate declarators.

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

Reviewed By: DavidTruby, klausler

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




More information about the All-commits mailing list