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

Add helper library for handcompiling benchmarks and tests #17

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

kyle-singer
Copy link
Collaborator

Adds a library (c2cilk) to the handcomp_tests such that it is easier to write "handcompiled" cilk code.
Certain features are written to make it a bit more annoying to use than otherwise could be achieved specifically because
we don't currently intend for this to be used outside of internal testing/benchmarking.

For internal benchmarking of opencilk, c2cilk makes it easier to write/port benchmarks to a handcompiled version such that we can test performance of cheetah in isolation of changes to the OpenCilk compiler. When compiled with -fopencilk, however, c2cilk will use the cilk_spawn, cilk_sync, etc., keywords, such that we can run roughly the same tests/benchmarks using the OpenCilk compiler.

I've tested the current handcomp_tests benchmarks on an x86_64 machine without avx, and there is no change in performance between the old handcompiled code and the handcompiled code using c2cilk.

@kyle-singer kyle-singer marked this pull request as draft September 16, 2024 18:35
@kyle-singer
Copy link
Collaborator Author

ABI changes require updates to this helper library (if we even want this still).

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.

1 participant