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
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).
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)