Hi, On 2018-03-23 00:42:39 +0000, Peter Eisentraut wrote: > Use stdbool.h if suitable > > Using the standard bool type provided by C allows some recent compilers > and debuggers to give better diagnostics. Also, some extension code and > third-party headers are increasingly pulling in stdbool.h, so it's > probably saner if everyone uses the same definition.
Hah. I was setting up LLVM buildfarm animals that run with LLVM assertions enabled, and was getting confused why they're failing after I'd run a check-world locally with assertions enabled. Turns out this broke it. I'll fix it. Just amazed that we've gone weeks without a change breaking my JIT work, and then within hours of it getting it got broken. Turns out LLVM represents a stdbool.h boolean as an i1 (int with 1 bit) rather than an i8 (int with 8 bits), which makes sense given the desired boolean behaviour (still takes 8 bits of storage). Greetings, Andres Freund
