-
Notifications
You must be signed in to change notification settings - Fork 310
allow to enable/disable standard library C++ modules in LLVM easyblock #3994
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
allow to enable/disable standard library C++ modules in LLVM easyblock #3994
Conversation
|
This doesn't really have a high priority, but was noticed by folks at JSC. |
|
It seems the default is to be ON also in LLVM 19 Anyway it is useful for this to be user-controllable will try to run a |
|
@boegelbot please test @ jsc-zen3 |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3580763465 processed Message to humans: this is just bookkeeping information for me, |
|
Test with Test report by @boegelbot Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (total: 3 hours 48 mins 1 secs) (1 easyconfigs in total) |
|
@boegelbot please test @ jsc-zen3 |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3581747062 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @Crivella Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (total: 7 hours 39 mins 44 secs) (1 easyconfigs in total) Option is being properly passed and not ignored |
|
Test report by @boegelbot Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (total: 3 hours 48 mins 25 secs) (1 easyconfigs in total) |
|
@boegelbot please test @ jsc-zen3 |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3582943959 processed Message to humans: this is just bookkeeping information for me, |
|
#3994 (comment) This has been running longer than i would expect |
|
|
@boegelbot please test @ jsc-zen3 |
Ah... guess having the bot report this kind of failures will be an improvement to be added |
|
@Thyre: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3585024586 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (total: 4 hours 57 mins 54 secs) (1 easyconfigs in total) |
Crivella
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Going in, thanks @Thyre! |
(created using
eb --new-pr)While LLVM always builds the standard library C++ modules, they are not installed by default for all versions. Marked as experimental (at least for LLVM 18 -- LLVM 20), the installation is only enabled by default in LLVM 20 and newer.
To still give users control about building the modules, add a configuration option. The default relies on the value set by LLVM.