Skip to content

Veclib flags not being passed to LTO flags #79564

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.

Sign up for GitHub

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

Jump to bottom
Closed
paschalis-mpeis opened this issue Jan 26, 2024 · 3 comments
Closed

Veclib flags not being passed to LTO flags #79564

paschalis-mpeis opened this issue Jan 26, 2024 · 3 comments
Labels
clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' release:backport release:merged
Milestone
LLVM 18.X Release

Comments

@paschalis-mpeis
Copy link
Member

We want to backport PR #78749 so the release has this feature.

The above PR converts clang's -fveclib flag to their equivalent name for opt's -vector-library,
so they are propagated correctly when LTO is used.
@paschalis-mpeis paschalis-mpeis added this to the LLVM 18.X Release milestone Jan 26, 2024
@github-project-automation github-project-automation bot moved this to Needs Triage in LLVM Release Status Jan 26, 2024
@paschalis-mpeis paschalis-mpeis added release:backport clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' labels Jan 26, 2024
@llvmbot
Copy link
Member

llvmbot commented Jan 26, 2024

@llvm/issue-subscribers-clang-driver

Author: Paschalis Mpeis (paschalis-mpeis)

We want to backport PR #78749 so the release has this feature.

The above PR converts clang's -fveclib flag to their equivalent name for opt's -vector-library,
so they are propagated correctly when LTO is used.

@github-actions github-actions bot mentioned this issue Jan 26, 2024
Closed
@github-actions GitHub Actions
Copy link

/pull-request #79566

@nikic nikic moved this from Needs Triage to Needs Merge in LLVM Release Status Jan 26, 2024
@tstellar
Copy link
Collaborator

Merged: fa0a72b

@tstellar tstellar moved this from Needs Merge to Done in LLVM Release Status Jan 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' release:backport release:merged
Projects
LLVM Release Status
Status: Done
Milestone
LLVM 18.X Release
Development

No branches or pull requests
4 participants
@tstellar @paschalis-mpeis @EugeneZelenko @llvmbot