Simplify intrinsics generation tool and regenerate intrinsics list#580

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

GuillaumeGomez wants to merge3 commits intorust-lang:master

base:master

Choose a base branch

fromGuillaumeGomez:regen-intrinsics

Open

Simplify intrinsics generation tool and regenerate intrinsics list#580

GuillaumeGomez wants to merge3 commits intorust-lang:masterfromGuillaumeGomez:regen-intrinsics

+535 −1,676

Conversation

Copy link

Member

GuillaumeGomez commentedDec 12, 2024

Fixes#575.

GuillaumeGomez requested a review fromantoyo

December 12, 2024 17:31

Copy link

MemberAuthor

GuillaumeGomez commentedDec 15, 2024

Interestingly enough, a lot of intrinsics used by rustc are not listed intblgen, which is very weird. So I think for now we can't really simplify the script...

Copy link

Contributor

antoyo commentedJan 10, 2025

Should we close this PR and the corresponding issue, then?

Copy link

MemberAuthor

GuillaumeGomez commentedJan 10, 2025

Likely not. I think we need to list intrinsics that should be generated manually because they're used by rust but not listed by LLVM. Maybe@bjorn3 got some new info about this situation (I suppose not but who knows?).