kuznia-rdzeni / coreblocks

RISC-V out-of-order core for education and research purposes
https://kuznia-rdzeni.github.io/coreblocks/
BSD 3-Clause "New" or "Revised" License
36 stars 14 forks source link

Run benchmarks on every PR #660

Closed xThaid closed 5 months ago

xThaid commented 5 months ago

The motivation is that performance changes should be visible by default (as opposed to manually triggering the benchmarks).

I am not adding benchmark label, as the job will fail. Most likely we need to merge this first, so permissions are propagated correctly. Here is the preview: https://github.com/xThaid/coreblocks/pull/2#issuecomment-2054193764

Fixes #657.

xThaid commented 5 months ago

I removed benchmark label as it will trigger the job that issues a comment in this PR, which will fail due to lack of permissions. I believe that putting this modified workflow on the master branch should solve permission issues.