Skip to content

Update expected instructions for LLVM 22#2007

Merged
folkertdev merged 4 commits into
rust-lang:mainfrom
nikic:ci-test
Jan 29, 2026
Merged

Update expected instructions for LLVM 22#2007
folkertdev merged 4 commits into
rust-lang:mainfrom
nikic:ci-test

Conversation

@nikic

@nikic nikic commented Jan 29, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@rustbot

rustbot commented Jan 29, 2026

Copy link
Copy Markdown
Collaborator

r? @folkertdev

rustbot has assigned @folkertdev.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@nikic

nikic commented Jan 29, 2026

Copy link
Copy Markdown
Contributor Author

First i686 difference is due to this: https://llvm.godbolt.org/z/oGW1d66zY

Codegen for _mm256_set_pd has changed (which is fine, it does not correspond to a specific instruction), though llvm-mca seems to think that the new form is slower. (I have some doubts that it's correct about that though.)

@nikic

nikic commented Jan 29, 2026

Copy link
Copy Markdown
Contributor Author

Second i686 difference: https://llvm.godbolt.org/z/41Td7oaqq

Codegen for _mm_set_ps changed, though here llvm-mca agrees that the new form is better.

@nikic

nikic commented Jan 29, 2026

Copy link
Copy Markdown
Contributor Author

The aarch64 failures are all variants of this: https://llvm.godbolt.org/z/PnGdxY3rf

Where zip2 is now used instead of a lane move.

@folkertdev

Copy link
Copy Markdown
Contributor

https://clang.godbolt.org/z/jhPP8hdPq

for powerpc I think the new output, actually emitting vrfin, is correct.

Comment thread .github/workflows/main.yml Outdated
nikic added 4 commits January 29, 2026 11:52
These don't correspond to specific instructions and will produce
different instructions on x86/x86_64 based on ABI details.
Otherwise this picks up vim .swp files.
It actually generates vrfin now
@nikic nikic changed the title Test CI after LLVM update Update expected instructions for LLVM 22 Jan 29, 2026
@nikic

nikic commented Jan 29, 2026

Copy link
Copy Markdown
Contributor Author

I filed llvm/llvm-project#178632 to check on the vperm2f128 change, as that seems the only potentially problematic one.

@folkertdev

Copy link
Copy Markdown
Contributor

Nice, thanks!

@folkertdev
folkertdev added this pull request to the merge queue Jan 29, 2026
Merged via the queue into rust-lang:main with commit a356d81 Jan 29, 2026
75 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants