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

Remove pre-compiled binaries #229

Merged
merged 1 commit into from
Apr 7, 2025
Merged

Remove pre-compiled binaries #229

merged 1 commit into from
Apr 7, 2025

Conversation

nikic
Copy link
Contributor

@nikic nikic commented Apr 4, 2025

For some reason, we have three pre-compiled binaries checked in for the gs, agrep and football benchmarks. These are not actually used (we'll build our own, as usual). Presumably they were committed by accident.

We also ship one more pre-compiled makeproto binary for the p2c benchmark. This is not necessary either, as we already ship the .proto files it generates.

@DavidSpickett
Copy link
Contributor

At least was here since the project was imported - a23d327.

Copy link
Contributor

@DavidSpickett DavidSpickett left a comment

Choose a reason for hiding this comment

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

LGTM.

I assume you used some command to search to see if there were any others?

For some reason, we have three pre-compiled binaries checked in for
the gs, agrep and football benchmarks. These are not actually used
(we'll build our own, as usual). Presumably they were committed by
accident.

We also ship one more pre-compiled makeproto binary for the p2c
benchmark. This is not necessary either, as we already ship the
`.proto` files it generates.
@nikic
Copy link
Contributor Author

nikic commented Apr 4, 2025

I assume you used some command to search to see if there were any others?

Nope, these were just flagged by some compliance tooling. But we remove some subdirectories for licensing reasons, so there are actually more... I did a more thorough check with find . -type d -name 'build*' -prune -o -type f -exec file {} + | grep "ELF.*executable" | cut -d: -f1 and dropped the genarch and makeproto binaries as well.

Worth noting that makeproto is different from the rest in that we don't seem to ship the source for it, but we do ship the .proto files it generates, so having the binary shouldn't be necessary either.

@nikic nikic merged commit 1cc11b0 into llvm:main Apr 7, 2025
1 check 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.

2 participants