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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
