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

[X86][Driver] Move mno-gather/mno-scatter from m_x86_Features_Group to m_Group. NFCI #65457

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

phoebewang
Copy link
Contributor

@phoebewang phoebewang commented Sep 6, 2023

m_x86_Features_Group always turn mno-xxxx into -target-feature -xxxx. In this case, we don't have -gather/-scatter but +prefer-no-gather/scatter.

This patch solves unexpected warning when using mno-gather/mno-scatter:

'-gather' is not a recognized feature for this target (ignoring feature)
'-scatter' is not a recognized feature for this target (ignoring feature)

…o m_Group. NFCI

m_x86_Features_Group always turn `mno-xxxx` into `-target-feature-xxxx`.
In this case, we don't have `-gather/-scatter` but `+prefer-no-gather/scatter`.
@phoebewang phoebewang requested a review from a team as a code owner September 6, 2023 09:23
@github-actions github-actions bot added the clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' label Sep 6, 2023
Copy link
Contributor

@XinWang10 XinWang10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, could work on my side.

@phoebewang phoebewang changed the title [X86][Driver] Move mno-gather/mno-scatter from m_x86_Features_Group yo m_Group. NFCI [X86][Driver] Move mno-gather/mno-scatter from m_x86_Features_Group to m_Group. NFCI Sep 6, 2023
@phoebewang phoebewang merged commit c5fabac into llvm:main Sep 6, 2023
2 checks passed
@phoebewang phoebewang deleted the main branch September 6, 2023 10:08
@phoebewang phoebewang restored the main branch September 6, 2023 10:09
avillega pushed a commit to avillega/llvm-project that referenced this pull request Sep 11, 2023
…o m_Group. NFCI (llvm#65457)

m_x86_Features_Group always turn `mno-xxxx` into `-target-feature
-xxxx`. In this case, we don't have `-gather/-scatter` but
`+prefer-no-gather/scatter`.

This patch solves unexpected warning when using
`mno-gather/mno-scatter`:
```
'-gather' is not a recognized feature for this target (ignoring feature)
'-scatter' is not a recognized feature for this target (ignoring feature)
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl'
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants