On 25.01.26 18:14, Tom Lane wrote:
Peter Eisentraut <[email protected]> writes:
On 23.01.26 23:18, Tom Lane wrote:
Hmm, yeah, their bug #70066 shows clearly that the __attribute__
spelling should work. But I think we'd better make the cutoff be
version 9 not version 6, because that same bug is quite clear
about when they fixed it. The lack of complaints from the buildfarm
may just indicate a lack of animals running the intermediate versions.
Ok, done that way.
Sigh ... that did not work. Various BF animals are now blowing up in
src/backend/jit/llvm because this macro definition breaks some usages
of alignas() in LLVM header files.
Maybe we could #define alignas this way for the two exposed usages
and then #undef afterwards?
Well, in C11, alignas is itself a macro (defined to _Alignas). I
suppose not in C++ though. That seems too tricky, though. I went with
your original proposal of disabling the affected typedefs on the
affected platform. That seems safest. These types aren't likely to be
used in extensions anyway, so this should have minimal practical impact.
