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

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 26 12:27:28 PDT 2022


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



More information about the cfe-commits mailing list