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

[leaderboard] Move leaderboard utility modules into compiler_gym namespace #161

Merged
merged 6 commits into from
Mar 31, 2021

Conversation

ChrisCummins
Copy link
Contributor

@ChrisCummins ChrisCummins commented Mar 30, 2021

This adds a compiler_gym.leaderboard module that contains the LLVM
codesize leaderboard helper code. New API docs provide improved
explanation of how to use it.

Fixes #158, #142.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 30, 2021
@ChrisCummins ChrisCummins changed the title [leaderboard] Move leaderboard utility into compiler_gym namespace. [leaderboard] Move leaderboard utility modules into compiler_gym namespace Mar 31, 2021
@ChrisCummins ChrisCummins force-pushed the eval-policy branch 3 times, most recently from 4ee6227 to f931dc7 Compare March 31, 2021 12:18
ChrisCummins and others added 4 commits March 31, 2021 14:18
This adds a compiler_gym.leaderboard module that contains the LLVM
codesize leaderboard helper code. New API docs provide improved
explanation of how to use it.

Issue #158.
This is to break the duplicate flag error from
//tests/benchmarks:parallelization_load_test.
Re-order the file so that leaderboard submissions appear directly
below pull requests. Then provide more details about the submission
review process.
Be clear that this leaderboard evaluates performance at reducing the
instruction count of LLVM-IR, not the binary codesize.
@ChrisCummins ChrisCummins merged commit 90b6690 into development Mar 31, 2021
@ChrisCummins ChrisCummins deleted the eval-policy branch March 31, 2021 14:36
@ChrisCummins ChrisCummins mentioned this pull request Mar 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move leaderboard helper code into compiler_gym namespace
2 participants