[clang] [C23] Fixed the value of BOOL_WIDTH (PR #117364)
Jens Gustedt via cfe-commits
cfe-commits at lists.llvm.org
Sat Nov 23 12:21:46 PST 2024
================
@@ -1103,7 +1103,15 @@ static void InitializePredefinedMacros(const TargetInfo &TI,
assert(TI.getCharWidth() == 8 && "Only support 8-bit char so far");
Builder.defineMacro("__CHAR_BIT__", Twine(TI.getCharWidth()));
- Builder.defineMacro("__BOOL_WIDTH__", Twine(TI.getBoolWidth()));
+ // The macro is specifying the number of bits in the value representation,
+ // not the number of bits in the object representation, which is what
+ // getBoolWidth() will return. For the bool/_Bool data type, there is only
----------------
gustedt wrote:
Yes, on modern architectures, `bool` and `_BitInt` should be the only integer types that have padding bits.
https://github.com/llvm/llvm-project/pull/117364
More information about the cfe-commits
mailing list