nikic wrote:

Looking at that buildbot, the triple it actually uses is powerpc-ibm-aix, even 
though the buildbot name is clang-ppc64-aix. And yet, for 32-bit AIX we have 
that size_t is long, which is the same size as int.

I think the right fix for the issue is to just drop the no-warning lines, as 
I've suggested here: 
https://github.com/llvm/llvm-project/pull/181800/changes#r2840874528 If the 
sizes don't match there should definitely be a warning. If they do match, then 
it depends on the exact type definitions, and we shouldn't test either way.

(Though at a higher level, if we ignore the immediate test issue here, IMHO, 
for this specific case of _Bool we should really be always warning regardless 
of what the underlying type definitions are. Using `_Bool` with `%zu` is both 
non-sensical and non-portable.)

https://github.com/llvm/llvm-project/pull/180566
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to