tra added inline comments.

================
Comment at: lib/Frontend/InitPreprocessor.cpp:1126
     Builder.defineMacro("__linux__");
+    if (AuxTriple.getEnvironment() == llvm::Triple::GNU)
+      Builder.defineMacro("__gnu_linux__");
----------------
AFAICT, we always define `__gnu_linix__` on Linux:
https://github.com/llvm-mirror/clang/blob/master/lib/Basic/Targets/OSTargets.h#L306

I think it should be the case here, too.


https://reviews.llvm.org/D51441



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to