Skip to content
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/16: Fix compiler-rt missing sanitizers when using useLLVM #267814

Closed
wants to merge 14 commits into from

Conversation

s1341
Copy link
Contributor

@s1341 s1341 commented Nov 16, 2023

Attempts to address #97688 by refactoring the llvm/16 toolchain build to make more sense.

The build order is now:

  1. {compiler-rt-no-libc}
  2. {compiler-rt-no-libc, libc}
  3. {compiler-rt-no-libc, libc, libc++}
  4. {compiler-rt-no-libc, libc, libc++, compiler-rt-libc}

The names of the stages have been updated to reflect this.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.11 Release Notes (or backporting 23.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

@s1341
Copy link
Contributor Author

s1341 commented Nov 16, 2023

Hrm. I don't know why it is infinite-recursioning. I don't have that issue locally. How can I reproduce what ofborg ran?

Copy link
Member

@Artturin Artturin Nov 16, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't reformat the file, it needs to be similar to the other llvm versions for diffing.

It's impossible to tell what changes you've made in it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry. it was accidental. I will fix it and push, but it'll have to be sunday.

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/3534

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/3648

Copy link
Member

@SuperSandro2000 SuperSandro2000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR rebuilds a lot of packages which means we must target staging. Please follow the contributing guide to not potentially ping a lot of people.

@github-actions github-actions bot added the 6.topic: jupyter Interactive computing tooling: kernels, notebook, jupyterlab label Mar 18, 2024
@s1341 s1341 changed the base branch from master to staging March 18, 2024 05:26
@github-actions github-actions bot removed 6.topic: python 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 6.topic: GNOME GNOME desktop environment and its underlying platform 6.topic: haskell 8.has: documentation This PR adds or changes documentation 8.has: changelog 8.has: module (update) This PR changes an existing module in `nixos/` 6.topic: emacs Text editor 6.topic: rust 6.topic: policy discussion 6.topic: golang 6.topic: vim 6.topic: ocaml 6.topic: stdenv Standard environment 6.topic: nodejs 6.topic: lua 6.topic: jupyter Interactive computing tooling: kernels, notebook, jupyterlab labels Mar 18, 2024
@SuperSandro2000
Copy link
Member

Locking this to avoid more pings because of the bad rebase. Please open a new PR.

@NixOS NixOS locked as too heated and limited conversation to collaborators Mar 19, 2024
@rrbutani rrbutani added the 6.topic: llvm/clang Issues related to llvmPackages, clangStdenv and related label May 27, 2024
@Ericson2314
Copy link
Member

#297144 is the successor PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants