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_11: link to the llvm compiler-rt share directory from the wrapper #100190

Merged
merged 1 commit into from Oct 11, 2020

Conversation

TredwellGit
Copy link
Member

Motivation for this change

See: #85925

@primeos Needed to enabled the important security feature for Chromium.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

This is required for features such as `-fsanitize=cfi` that (by default) load the file `…/resource-root/share/cfi_blacklist.txt`.
@primeos
Copy link
Member

primeos commented Oct 11, 2020

Thanks :)

@TredwellGit TredwellGit deleted the llvm_11 branch October 11, 2020 22:05
primeos pushed a commit to primeos/nixpkgs that referenced this pull request Nov 4, 2020
NixOS#100190)

A port of NixOS#85925 for LLVM 11 to enable CFI for Chromium.

This is required for features such as `-fsanitize=cfi` that (by default)
load the file `…/resource-root/share/cfi_blacklist.txt`.

(cherry picked from commit 03dd1b3)
rvem pushed a commit to serokell/nixpkgs that referenced this pull request Nov 8, 2022
NixOS#100190)

A port of NixOS#85925 for LLVM 11 to enable CFI for Chromium.

This is required for features such as `-fsanitize=cfi` that (by default)
load the file `…/resource-root/share/cfi_blacklist.txt`.

(cherry picked from commit 03dd1b3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants