[PATCH] D62202: Work around a Visual C++ bug

Tue May 21 08:35:16 PDT 2019

probinson added a comment.

To provide some missing details:
The original source gets a bogus compile-time error with Visual Studio 2017, prior to version 15.8.  I have 15.2, so I need this patch to build Clang.
Our current minimum-version requirement (per CheckCompilerVersion.cmake) is Visual Studio 2015 (some specific update), which assumes all Visual Studio 2017 versions are usable.  So by that criterion, we need this patch for all Visual Studio 2017 versions to be usable.
Arguably the anonymous-namespace version is the "more modern" tactic anyway, so as a long-term fix this is actually fine anyway.

My hesitation is... it *is* a compiler bug, and I'm working around it... wondered what y'all think about it.

Repository:
  rC Clang

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

https://reviews.llvm.org/D62202