[PATCH] D85937: [flang][msvc] Split class declaration and constexpr variable definition. NFC.

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 14 13:36:52 PDT 2020


Meinersbur added a comment.

In D85937#2219028 <https://reviews.llvm.org/D85937#2219028>, @klausler wrote:

> You might have to use conditional preprocessing to make this workaround specific to MSVC.

The workaround is valid C++ and works fine with every compiler. So why add a #if maze?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85937/new/

https://reviews.llvm.org/D85937



More information about the llvm-commits mailing list