Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clang -O4 Build Fails H5Tconv.c #4208

Closed
hyoklee opened this issue Mar 21, 2024 · 4 comments
Closed

Clang -O4 Build Fails H5Tconv.c #4208

hyoklee opened this issue Mar 21, 2024 · 4 comments
Assignees
Labels
Component - Build CMake, Autotools Component - C Library Core C library issues (usually in the src directory) Priority - 0. Blocker ⛔ This MUST be merged for the release to happen
Milestone

Comments

@hyoklee
Copy link
Member

hyoklee commented Mar 21, 2024

Please check the new _Float16 code with the highest optimization levels for all compilers.

  "___truncxfhf2", referenced from:
      _H5T__conv_ldouble__Float16 in H5Tconv.c.o
ld: symbol(s) not found for architecture x86_64

See also: https://my.cdash.org/viewBuildError.php?buildid=2524289

@hyoklee hyoklee added Priority - 0. Blocker ⛔ This MUST be merged for the release to happen Component - C Library Core C library issues (usually in the src directory) Component - Build CMake, Autotools labels Mar 21, 2024
@hyoklee hyoklee added this to the 1.14.4 milestone Mar 21, 2024
@jhendersonHDF
Copy link
Collaborator

Was clang installed with homebrew on that machine? See JuliaLang/julia#52067 which is almost certainly related.

@hyoklee
Copy link
Member Author

hyoklee commented Mar 21, 2024

I'm using built-in Apple /usr/bin/clang.

@chenrui333
Copy link

@jhendersonHDF I am still seeing the build failure with the 12 builds (full build log, https://github.com/Homebrew/homebrew-core/actions/runs/8973996030/job/24648320497), can I get some help on it?

relates to Homebrew/homebrew-core#170959

@jhendersonHDF
Copy link
Collaborator

Hi @chenrui333,

several versions of AppleClang have been problematic due to having the type and macros for _Float16 support while (I believe) having an issue with the version of compiler-rt used. If you're not necessarily interested in _Float16 support, it can be disabled with the configure option --disable-nonstandard-feature-float16 (Autotools) or -DHDF5_ENABLE_NONSTANDARD_FEATURE_FLOAT16=OFF (CMake). Otherwise, we can see if we can come up with a configure time check that will fully catch this on MacOS 12. I believe we were only using homebrew to install ninja on the MacOS 12 GitHub runner at the time this was tested, so there may be some differences between what was being tested.

While this particular GitHub issue dealt with compile failures in the library's main source code, it looks like your configure logs show that the library correctly picked up and built in _Float16 support without failing any of the configure checks I added, but then fails when compiling the test code. I'm not sure why that would be.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Build CMake, Autotools Component - C Library Core C library issues (usually in the src directory) Priority - 0. Blocker ⛔ This MUST be merged for the release to happen
Projects
None yet
Development

No branches or pull requests

3 participants