Skip to content

release/19.x: [OpenMP] Create versioned libgomp softlinks (#112973) #115944

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

Closed
wants to merge 1 commit into from

Conversation

llvmbot
Copy link
Member

@llvmbot llvmbot commented Nov 12, 2024

Backport eccdb24

Requested by: @ye-luo

Add libgomp.1.dylib for MacOS and libgomp.so.1 for Linux

Linkers on Mac and Linux pick up versioned libgomp dynamic library
files. The existing softlinks (libgomp.dylib for MacOS and libgomp.so
for Linux) are insufficient. This helps alleviate the issue of mixing
libgomp and libomp at runtime.

(cherry picked from commit eccdb24)
@llvmbot llvmbot added this to the LLVM 19.X Release milestone Nov 12, 2024
@llvmbot
Copy link
Member Author

llvmbot commented Nov 12, 2024

@shiltian What do you think about merging this PR to the release branch?

@llvmbot llvmbot requested a review from shiltian November 12, 2024 21:54
@llvmbot llvmbot added the openmp:libomp OpenMP host runtime label Nov 12, 2024
@shiltian
Copy link
Contributor

Are we still releasing 19?

@ye-luo
Copy link
Contributor

ye-luo commented Nov 12, 2024

Are we still releasing 19?

yes. Point releases 19.1.x

@tru
Copy link
Collaborator

tru commented Nov 15, 2024

Hmm. Is this really a regression? Which is what we usually only fix at this point. It seems like a "feature". What's the risk of this creating problems for users already adopted 19.x?

@ye-luo
Copy link
Contributor

ye-luo commented Nov 15, 2024

Hmm. Is this really a regression? Which is what we usually only fix at this point. It seems like a "feature". What's the risk of this creating problems for users already adopted 19.x?

It is not a regression but a bug fix. The issue has been there for years. NVHPC ships the similar softlinks to their OpenMP runtime for years. It should be low risk but I’ll let you decide to apply the change to 19.x or not.

@tru
Copy link
Collaborator

tru commented Nov 19, 2024

I think it makes more sense to do this change in 20.x instead of 19.x.

@ye-luo
Copy link
Contributor

ye-luo commented Nov 20, 2024

I think it makes more sense to do this change in 20.x instead of 19.x.

I can live with that.

@ye-luo ye-luo closed this Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
openmp:libomp OpenMP host runtime
Projects
Development

Successfully merging this pull request may close these issues.

4 participants