[PATCH] D132754: [clang] Add __is_target_variant_{os,environment} builtins

[Nico Weber via Phabricator via cfe-commits]

thakis added inline comments.


================
Comment at: clang/lib/Lex/PPMacroExpansion.cpp:1713
               // FIXME: This is inconsistent; we usually suggest detecting
               // builtin macros via #ifdef. Don't add more cases here.
               .Case("__is_target_arch", true)
----------------
(I saw this comment, but system headers force our hand.)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132754/new/

https://reviews.llvm.org/D132754

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