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

Simplify generate_intrinsics.py #575

Open
bjorn3 opened this issue Dec 6, 2024 · 0 comments · May be fixed by #580
Open

Simplify generate_intrinsics.py #575

bjorn3 opened this issue Dec 6, 2024 · 0 comments · May be fixed by #580
Labels
enhancement New feature or request

Comments

@bjorn3
Copy link
Member

bjorn3 commented Dec 6, 2024

Currently it tries to fetch data from three sources:

The last two sources are because the last commit on llvm-project may have removed some intrinsics that stdarch uses. It should be possible to simplify this to just Intrinsics.td from llvm-project by picking the exact commit that the nightly version against which cg_gcc will build uses as LLVM version. If any intrinsics are removed there, stdarch has had to stop using them before, so the removal of the intrinsic wouldn't matter.

The manual parsing of llvm-tblgen output could also likely be replaced by usage of the --dump-json flag.

@bjorn3 bjorn3 added the enhancement New feature or request label Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant