[PATCH] D117238: [C2x] Add BITINT_MAXWIDTH support

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 25 12:12:57 PST 2022


erichkeane added inline comments.


================
Comment at: clang/lib/Frontend/InitPreprocessor.cpp:917
 
+  Builder.defineMacro("__BITINT_MAXWIDTH__", Twine(TI.getMaxBitIntWidth()));
+
----------------
Could you do an assert that TI.getMaxBitIntWidth() is not greater than the LLVM maximum (just somewhere, here seemed to make the most sense).


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

https://reviews.llvm.org/D117238



More information about the cfe-commits mailing list