programmerjake requested changes to this revision.
programmerjake added inline comments.
This revision now requires changes to proceed.
Comment at: clang/lib/Frontend/InitPreprocessor.cpp:589
+if (LangOpts.CPlusPlus) {
+ Builder.defineMacro("__GNUG__", Twine(GNUCMinor));
programmerjake added a comment.
The `__GNUG__` macro is defined to be 4 rather than matching `__GNUC__`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68055/new/
https://reviews.llvm.org/D68055
___
cfe-
programmerjake added a comment.
Shouldn't `__GNUG__` match `__GNUC__`?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68055/new/
https://reviews.llvm.org/D68055
___
cfe-commits mailing list
cfe-commits@
