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

Refactor libCEED QFunction source code to accelerate JIT compilation #180

Merged
merged 3 commits into from
Feb 5, 2024

Conversation

sebastiangrimberg
Copy link
Contributor

This helps NVRTC/HIP RTC in libCEED by compiling only the source code needed for the QFunction of a given dimension. No new functionality is added here, the only change is reorganizing existing code across files.

@sebastiangrimberg sebastiangrimberg added the performance Related to performance label Feb 2, 2024
Base automatically changed from sjg/docs-typos to main February 2, 2024 21:31
@sebastiangrimberg sebastiangrimberg force-pushed the sjg/libceed-qfunction-source branch 2 times, most recently from 4a464e8 to 300d3bf Compare February 2, 2024 22:41
@sebastiangrimberg sebastiangrimberg changed the base branch from main to sjg/pa-default-dev February 2, 2024 22:43
Copy link
Collaborator

@hughcars hughcars left a comment

Choose a reason for hiding this comment

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

Found a few missed closing header comments.

Base automatically changed from sjg/pa-default-dev to main February 5, 2024 16:40
@sebastiangrimberg sebastiangrimberg force-pushed the sjg/libceed-qfunction-source branch from 6a97804 to 538e8d1 Compare February 5, 2024 16:41
@sebastiangrimberg sebastiangrimberg merged commit 3d821aa into main Feb 5, 2024
17 checks passed
@sebastiangrimberg sebastiangrimberg deleted the sjg/libceed-qfunction-source branch February 5, 2024 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Related to performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants