[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