[PATCH] D51441: Add predefined macro __gnu_linux__ for proper aux-triple

Yaxun Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 29 13:17:29 PDT 2018


yaxunl added a comment.

In https://reviews.llvm.org/D51441#1218034, @yaxunl wrote:

> In https://reviews.llvm.org/D51441#1218010, @tra wrote:
>
> > While we're here, perhaps `Builder.defineMacro("__linux__")` should be changed to `DefineStd("linux")` which defines `linux/__linux/__linux__`?
>
>
> Will do when committing. Thanks.


Sorry DefineStd is a function not available in InitPreprocessor.cpp. To reuse that function refactoring is required, which is out of scope for this patch.


https://reviews.llvm.org/D51441





More information about the cfe-commits mailing list