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