llvm-{5,6}: unconditionally set triples#42454
Conversation
|
I guess slight addendum is I think |
|
Sure! I'm more concerned about this being "right" in various scenarios I may not have tried/thought of/etc. :). I'll split off a PR for just LLVM 4 and reduce this to 5/6, coming soon... :) |
Previously we only set these when using musl. See NixOS#42452 (comment) Same should be done for changes in NixOS#42452, I'll add a follow-up commit doing so in the PR containing this or wherever makes sense-- depending on how review/merging of that goes. Sending similar changes for LLVM 4 to master, as suggested.
b581e6a to
7088489
Compare
Split off from NixOS#42454 as requested.
Follow-on to same changes made to other versions: See NixOS#42454, NixOS#42456.
|
@GrahamcOfBorg build tests.cc-wrapper-clang-5 tests.cc-wrapper-libcxx-5 |
|
Success on x86_64-linux (full log) Attempted: tests.cc-wrapper-clang-5, tests.cc-wrapper-libcxx-5 Partial log (click to expand)
|
|
Failure on x86_64-linux (full log) Attempted: tests.cc-wrapper-clang-6, tests.cc-wrapper-libcxx-6 Partial log (click to expand)
|
|
Failure on x86_64-darwin (full log) Attempted: tests.cc-wrapper-clang-5, tests.cc-wrapper-libcxx-5 Partial log (click to expand)
|
|
Failure on aarch64-linux (full log) Attempted: tests.cc-wrapper-clang-5, tests.cc-wrapper-libcxx-5 Partial log (click to expand)
|
|
Success on aarch64-linux (full log) Attempted: tests.cc-wrapper-clang-6, tests.cc-wrapper-libcxx-6 Partial log (click to expand)
|
|
Failure on x86_64-darwin (full log) Attempted: tests.cc-wrapper-clang-6, tests.cc-wrapper-libcxx-6 Partial log (click to expand)
|
|
Well despite what this says, these all work on at least x86_64-linux in my testing. |
|
As mentioned on the PR for llvm 4 with these changes, things look okay so far--and since this goes to staging merging this as well. LMK if there are any problems, will try to check hydra across the next few days. |
Previously we only set these when using musl.
See #42452 (comment)
Same should be done for changes in #42452,
I'll add a follow-up commit doing so in the PR containing this
or wherever makes sense-- depending on how review/merging of that goes.
Sending to staging since these later LLVM versions are used in many
places, in particular are the basis of Darwin stdenv :).
Building/testing still underway, help appreciated since this is fairly
wide-reaching and I can only test so much (and can't test Darwin at all).
sandboxinnix.confon non-NixOS)nix-shell -p nox --run "nox-review wip"./result/bin/)nix path-info -Sbefore and after)