Skip to content

V5.0.x: configury: add support for LLVM 19 #12514

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

Merged
merged 3 commits into from
May 2, 2024
Merged

Conversation

janjust
Copy link
Contributor

@janjust janjust commented May 2, 2024

No description provided.

LLVM 18 supports the ASYNCHRONOUS keyword in an interface,
but not in a subroutine (!) so test both in order to avoid
a false positive that will lead to compilation failure.

Signed-off-by: Gilles Gouaillardet <[email protected]>
(cherry picked from commit 8ef1ef8)
as reported in open-mpi#12506, upcoming LLVM 19 can generate some
false positive that will make the mpi Fortran modules unusable.
Harden the test by using a module in order to fix that.

Thanks Paul Osmialowski for bringing this to our attention.

Signed-off-by: Gilles Gouaillardet <[email protected]>
(cherry picked from commit 8c601e1)
Try the !DIR$ IGNORE_TKR directive and the type(*)
type in order to support upcoming LLVM 19 flang compiler.

Thanks Paul Osmialowski for the report and suggesting the solution.

Signed-off-by: Gilles Gouaillardet <[email protected]>
(cherry picked from commit 0bc995f)
@janjust janjust requested a review from jsquyres May 2, 2024 16:23
@github-actions github-actions bot added this to the v5.0.4 milestone May 2, 2024
@janjust
Copy link
Contributor Author

janjust commented May 2, 2024

@ggouaillardet @jsquyres cherry-pick for #12512

@janjust janjust merged commit a948e27 into open-mpi:v5.0.x May 2, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants