-
Notifications
You must be signed in to change notification settings - Fork 766
LLVM: Add patch to fix bindc commonblocks in fortran compilation #24495
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
Conversation
|
@boegelbot please test @ jsc-zen3 |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 3506560029 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
Thyre
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should test if this is sufficient for HDF5, but looks good to me otherwise.
Already did on both 2023b and 2025b, will open more PRs tomorrow from work. |
|
Great! I also have a few more things in my local repository (HPL, HPCG with EasyBlock changes, GROMACS, MUST, OTF-CPT and Score-P). I'll probably look into other stuff at some point as well, but those had the highest priority for me first. |
|
Going to re-run a build without |
|
@boegelbot please test @ jsc-zen3 |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 3511084472 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
Micket
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, did you guys manage to verify that it fixed the issue with hdf5?
@Crivella already verified this, but I think forgot to upload the test report. I have two builds of LLVM running and will verify with HDF5 afterwards. |
|
Test report by @Thyre |
|
@Micket, see #24516 (comment) for a successful test report for HDF5 with this PR. |
|
Test report by @Thyre |
Applies patch from
to LLVMs before 21 aimed at being the base of a toolchain.
Needed EG to compile HDF5 with LLVM and avoid the error described in
https://discourse.llvm.org/t/hlfir-declare-op-of-numeric-logical-or-assumed-type-entity-must-not-have-length-parameters/83629