Use LLVM easyblock for Clang 18 easyconfigs#23055
Use LLVM easyblock for Clang 18 easyconfigs#23055Crivella merged 9 commits intoeasybuilders:developfrom
Conversation
|
@boegelbot please test @ jsc-zen3-a100 |
|
@boegel: 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 3033481563 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Test report by @boegel |
|
@Crivella It does not. It simply downloads the easyblock as-is and includes the path to the downloaded location. So only one of them is used, which one depends on the order we add the new paths. I don't think merging is even possible: How would you handle conflicts (when you download diffs) and building from a merged PR? |
|
Yeah I assumed so, but also deleted the comment as i noticed it was 2 separate ones clang/llvm and not llvm/llvm my bad there |
|
Test report by @Flamefire |
|
Test report by @Flamefire |
|
Test report by @Flamefire |
|
Test report by @Flamefire |
|
Test report by @Flamefire |
|
The tests fail mostly with: The 2nd seems to be a known issue with Clang 18, the other is weird: Their test tool LLVM 20 works. Looks like it is supposed to add an rpath, but can't find anything relevant in the logs. Maybe we should keep skipping the tests? |
|
We are gonna need the same patches that are applied for LLVM to pass all the tests |
|
In particular for the This is needed to ensure LD_LIBRARY_PATH is passed when building the tests where needed. See |
|
Test report by @Flamefire |
|
Test report by @Flamefire |
|
Test report by @Flamefire |
|
Ok, now I can't reproduce the failures anymore. |
The sanitizer tests can be flaky... considering you had 11/10 failures with 3 from the sanitizers i am not too surprised |
|
So add "stack-overflow-with-asan.test" to the ignore list and call it a day? The error from
and |
|
Considering in the newer ECs we do EG: |
|
@boegelbot please test @ jsc-zen3-a100 |
|
@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 3132130925 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @Flamefire |
|
Test report by @Flamefire |
|
Test report by @boegelbot |
|
Umh we might also want to disable the fuzzer tests, when we set Right now through Concerning the failing CUDA tests, not sure how relevant they are or if we want to add them to ignores tests in the EC ... |
|
The I also tried enabling the tests for the original EasyConfig (using Clang easyblock) but there it couldn't even finish compiling. Looks like a concurrency issue. |
|
@boegelbot please test @ jsc-zen3-a100 |
|
@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 3139036781 processed Message to humans: this is just bookkeeping information for me, |
|
I think those failures aren't serious after all: Calling Comparing the EC built with Clang and LLVM and running the simple example:
So good to ignore I guess |
|
Test report by @Flamefire |
|
Test report by @boegelbot |
|
@boegelbot please test @ jsc-zen3 |
|
Doing one final test building all the software that currently exists in EB that depends on |
|
@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 3149606457 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
Crivella
left a comment
There was a problem hiding this comment.
LGTM
passes both the CPU/GPU builds and also tested building software that depends on Clang-18.1.8-GCCcore-13.3.0 both as a builddep and normal dep
|
Going in, thanks @Flamefire! |
Pulled out of #23028
Requires:
Comparing the build with this and previous Clang easyblock shows that this fixes installing files for every single GPU architecture instead of only the intended ones, see easybuilders/easybuild-easyblocks#3755 (comment)
Also enabling testing as suggested in Slack and related issues.