llvmPackages*.llvm: Disable checkPhase on powerpc64-linux #307819
Merged
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.
Attempt number two, #296666. 😓
CC @NixOS/llvm
Many failures, mostly in MCJIT & Orc(more JIT).
LLVM 9
Failing Tests (125):
llvm-9.0.1.log
LLVM 17
Failed Tests (91):
llvm-17.0.6.log
Notes:
Clang-based stdenv *mostly works*, though it'll need some help from our cc-wrapper
LLVM/Clang ignores the
gnuabielfv{1,2}
ABI part of the config triplet and culls it down to justgnu
:GNUABIELFV1
/GNUABIELFV2
values/llvm/include/llvm/TargetParser/Triple.h:230-242
StartsWith("gnu", Triple::GNU)
catches the otherwise-unhandledgnuabielfv{1,2}
and reduces it to just"gnu"
/llvm/lib/TargetParser/Triple.cpp:622-636
/llvm/include/llvm/TargetParser/Triple.h:906-912
Which results in the compiler producing ELFv1 objects, even though we're asking for ELFv2 ones. So most linking will fail.
When running with
-v
:Relevant part:
Planning to add something like this to our cc-wrapper to work around this, which gets
hello
building withllvmPackages_17.stdenv
:But we can figure out the details of that in a separate PR.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.