Skip to content

Conversation

@GuillaumeGomez
Copy link
Member

No description provided.

"sse41.ptestnzc" => "__builtin_ia32_ptestnzc128",
"sse41.ptestz" => "__builtin_ia32_ptestz128",
"sse41.round.pd" => "__builtin_ia32_roundpd",
"sse41.round.ps" => "__builtin_ia32_roundps",
Copy link
Member Author

Choose a reason for hiding this comment

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

Seems like this intrinsic still exist. Weird. Well, moving it to the "old archs" then.

Copy link
Contributor

@antoyo antoyo Jan 16, 2026

Choose a reason for hiding this comment

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

Shouldn't all of them have been moved in the previous PR?

Copy link
Member Author

Choose a reason for hiding this comment

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

No, it's the LLVM update which changed that. I wonder if that's because we use an older LLVM version...

@GuillaumeGomez
Copy link
Member Author

Gonna wait for you to re-approve just in case. ;)

@antoyo
Copy link
Contributor

antoyo commented Jan 16, 2026

Gonna wait for you to re-approve just in case. ;)

All good. You can merge.

@GuillaumeGomez GuillaumeGomez merged commit 9b2d8e5 into rust-lang:master Jan 16, 2026
38 checks passed
@GuillaumeGomez GuillaumeGomez deleted the regen-intrinsics branch January 16, 2026 21:32
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.

2 participants