POSIX UINT16_C (and other) macros are incorrect in stdint.h

[kallisti5]

LLVM doesn't build on x86_64. Tracked it down to a bug in our stdint.h. I found the solution when I ran onto a DragonflyBSD post seeing the same LLVM build error.

​http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/2a61a4766f203ace8a0d0ed677e146bd3959dd58

That shows a fix for the same issue present in our stdint.h

Working on a patch now, but need to do some in-depth testing before I commit it.

[kallisti5]

I also should note that I see the same type defined in the following headers:

• src/bin/coreutils/lib/stdint.h
• src/bin/coreutils/lib/stdint.in.h
• src/bin/findutils/gnulib/lib/stdint.h
• src/bin/findutils/gnulib/lib/stdint.in.h
• src/bin/network/wget/lib/stdint.in.h
• src/bin/network/wget/md5/stdint.in.h

Need to check those projects to see if they also updated that headers.

[kallisti5]

Found the core issue.

headers/config/types.h:

If platform is 64-bit, uint64 is signed long If platform is not 64-bit, uint64 is signed long long

Our stdint.h UINT64_C defines need adjusted to match this logic.

[kallisti5]

untested rev 1

[korli]

Which LLVM version?

[pulkomandy]

wget and findutils should be moved to packages, anyway. Maybe coreutils could be moved as well.

[kallisti5]

Replying to korli:

Which LLVM version?

LLVM 3.4 (latest) I never actually tried building llvm on x86_64, so i'm guessing this bug has always existed.

working through testing the changes above now. Hopefully i'll have enough time to try an llvm build this evening before commiting the modifications.

[korli]

Replying to kallisti5:

Replying to korli:

Which LLVM version?

LLVM 3.4 (latest) I never actually tried building llvm on x86_64, so i'm guessing this bug has always existed.

Weirdly haikuporter builds only version 3.2 of llvm instead of 3.4 on x86_64 when asking for a mesa 10.0.2 build. Hence why this problem wasn't met before.

[kallisti5]

resolved as of hrev46902. Confirmed in llvm that they moved from using 1ull to UINT64_C(1) recently uncovering this issue.

LLVM 3.4 compiled successfully as did x86_64 Haiku.

[bonefish]

Replying to pulkomandy:

wget and findutils should be moved to packages, anyway. Maybe coreutils could be moved as well.

Absolutely. Cf. #10267.