-
Notifications
You must be signed in to change notification settings - Fork 62
Rust and cuda clang support (#1873) #1881
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
20a5ca8 to
02293a7
Compare
Member
Author
|
Ok, the blocking issue here seems to be around permissions to create files/folders in the runner |
27cdb0b to
c9052f2
Compare
Open
7f91563 to
373cffa
Compare
* Switch compiler to clang (not portable) * test add_num * compile with llvm tools * not working linking * not fixed * Update ex1-volume.h * update * remove global path * changes * changes 2 * crate works * basic gpu rust compilation * still not working * rust source roots basic support * nvrtc/clang selection * cleanup * update example (not working) * add rust example * fix merge issue * delete temp files * cleanup * rust qfunc 2d array (needs doc) * cleanup * more cleanup * downgrade back to c++11 * format * final draft cleanup * formatting + CUDA_CLANG -> GPU_CLANG * Update cuda CEED_QFUNCTION_RUST * fix python * fix python and format * format fr * update comment * fix python fr * Apply error suggestions from code review * update errors to libceed format * Apply suggestions from code review * add optimization flag * remove line breaks * Apply suggestions from code review * avoid python in macro better * add rust example * format * Apply suggestions from code review * move rust example to own directory * Simplify python exclusion logic * re-fix python * Update python/build_ceed_cffi.py * change names and simplify makefile * Revert "change names and simplify makefile" This reverts commit 96e762f. * Apply Jeremy's diff * Simplify CeedCallSystem * use rust-install llvm tools * add gitignores * update paths * example absolute path * fix comments * apply partial diff * add newline * add makefile * Update examples/rust-qfunctions/Makefile * update makefile --------- Co-authored-by: Allen MacFarland <alma4974@noether>
373cffa to
f03c7ee
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a staging branch for #1873 so we can run Noether CI and fix any issue there before doing final approval and merging