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

Fix build for quartz, newer clang versions, optional CUDA #128

Merged
merged 7 commits into from
Jan 19, 2024

Conversation

slabasan
Copy link
Collaborator

@slabasan slabasan commented Jan 12, 2024

Fixes #127, #129 and #49.

@tpatki
Copy link
Member

tpatki commented Jan 18, 2024

@slabasan Shall we merge this in before our quartz hackathon, or do we need to test this more extensively?

@slabasan slabasan changed the title Building on x86 Fix build for quartz Jan 19, 2024
@slabasan slabasan changed the title Fix build for quartz Fix build for quartz, newer clang versions, optional CUDA Jan 19, 2024
@slabasan slabasan merged commit 17c4e54 into flux-framework:main Jan 19, 2024
5 checks passed
alizalisan pushed a commit to alizalisan/PerfFlowAspect that referenced this pull request Jan 24, 2024
…work#128)

* Initial changes for PFA on quartz

* Add check for CallSite.h and AbstractCallSite.h based on clang version

* set build variables based on clang version

* add LLVM_DIR to lassen and quartz host configs

* cuda optional dependency

* update name of quartz hostconfig

---------

Co-authored-by: Tapasya Patki <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants