cc-wrapper: add libcxxabi include flag for LLVM by natto1784 · Pull Request #255488 · NixOS/nixpkgs

natto1784 · 2023-09-16T14:30:00Z

Description of changes

Should fix #255487

Things done

Added -isystem ${libcxxabi}/include/c++/v1 to libcxx-cxxflags for clang

Artturin · 2023-09-17T21:37:12Z

pkgs/test/cc-wrapper/default.nix

This pr fixes this test for tests.cc-wrapper.llvmTests.llvmPackages_{15,16}.libcxx

Artturin · 2023-09-17T21:37:25Z

pkgs/development/compilers/llvm/16/default.nix

removed old workaround

Artturin · 2023-09-17T21:39:02Z

I have to consider whether we should do this or

install -m 644 ../../${pname}/include/__cxxabi_config.h "$dev/include" in libcxxabi postInstall, were already copying cxxabi.h in there

Artturin · 2023-09-17T22:12:43Z

pkgs/build-support/cc-wrapper/default.nix

Before llvm 15 __cxxabi_config.h was included in libcxx so this worked correctly

llvmPackages_13.libcxx.dev 3,133 r /nix/store/5wfncnq3svyf54hfahi44smavyfbhq5b-libcxx-13.0.1-dev/include/c++/v1/__cxxabi_config.h llvmPackages_7.libcxx.dev 2,278 r /nix/store/2i745a6pvd2qbi0lf9a61ia0svvg9q0b-libcxx-7.1.0-dev/include/c++/v1/__cxxabi_config.h llvmPackages_14.libcxx.dev 3,142 r /nix/store/jri1wy9fxnnln4wv7dmzdhs9w5ll4jfa-libcxx-14.0.6-dev/include/c++/v1/__cxxabi_config.h llvmPackages_15.libcxxabi.dev 3,266 r /nix/store/bbvhnvq66lr6yzqxqbzff2341h9xg332-libcxxabi-15.0.7-dev/include/c++/v1/__cxxabi_config.h llvmPackages_6.libcxx.dev 2,032 r /nix/store/bpyj2sk6mh9vlli539jsr0w45rpig5pc-libcxx-6.0.1-dev/include/c++/v1/__cxxabi_config.h llvmPackages_5.libcxx.dev 2,032 r /nix/store/fciabccmn17ijmfxxn50diin0dcjz7m6-libcxx-5.0.2-dev/include/c++/v1/__cxxabi_config.h llvmPackages_8.libcxx.dev 2,278 r /nix/store/8sg9d3j38i3p13sg7pp4bc44szqakj7s-libcxx-8.0.1-dev/include/c++/v1/__cxxabi_config.h llvmPackages_9.libcxx.dev 2,411 r /nix/store/1b5yn18w09n9r50i9162zvn3zq3jn98k-libcxx-9.0.1-dev/include/c++/v1/__cxxabi_config.h llvmPackages_16.libcxxabi.dev 3,266 r /nix/store/02wpjmp2zjjxz13z7g599mniwi25zkcy-libcxxabi-16.0.6-dev/include/c++/v1/__cxxabi_config.h llvmPackages_10.libcxx.dev 2,500 r /nix/store/j41yp16y29b7gw2bd5c1ljfyyykgsh12-libcxx-10.0.1-dev/include/c++/v1/__cxxabi_config.h llvmPackages_12.libcxx.dev 3,133 r /nix/store/fyr084bq8496099nllfjzapis7mxpj39-libcxx-12.0.1-dev/include/c++/v1/__cxxabi_config.h

and
cxxabi.h was in both libcxx and libcxxabi

llvmPackages_13.libcxx.dev 7,215 r /nix/store/5wfncnq3svyf54hfahi44smavyfbhq5b-libcxx-13.0.1-dev/include/c++/v1/cxxabi.h llvmPackages_7.libcxx.dev 6,945 r /nix/store/2i745a6pvd2qbi0lf9a61ia0svvg9q0b-libcxx-7.1.0-dev/include/c++/v1/cxxabi.h llvmPackages_14.libcxx.dev 7,215 r /nix/store/jri1wy9fxnnln4wv7dmzdhs9w5ll4jfa-libcxx-14.0.6-dev/include/c++/v1/cxxabi.h libcxxrt.out 9,040 r /nix/store/sdw3q6nzxgdh6n6jdnnsgm7jpg5wsp1v-libcxxrt-unstable-2022-08-08/include/cxxabi.h llvmPackages_12.libcxxabi.dev 7,215 r /nix/store/6lzf327lxdi743i8n8s3igq962jcp5r7-libcxxabi-12.0.1-dev/include/cxxabi.h llvmPackages_15.libcxxabi.dev 7,237 r /nix/store/bbvhnvq66lr6yzqxqbzff2341h9xg332-libcxxabi-15.0.7-dev/include/c++/v1/cxxabi.h llvmPackages_15.libcxxabi.dev 7,237 r /nix/store/bbvhnvq66lr6yzqxqbzff2341h9xg332-libcxxabi-15.0.7-dev/include/cxxabi.h llvmPackages_5.libcxxabi.dev 6,945 r /nix/store/qlzdwhhd0x9fda4mk1a83jmmxza4a64p-libcxxabi-5.0.2-dev/include/cxxabi.h libcxxabi.dev 7,029 r /nix/store/h84lr9b4q34f37jvqzqkkf4gnrlrgahl-libcxxabi-11.1.0-dev/include/cxxabi.h llvmPackages_14.libcxxabi.dev 7,215 r /nix/store/zkfgj0hy0za0yhwnrfdndvzr301aj4pq-libcxxabi-14.0.6-dev/include/cxxabi.h llvmPackages_6.libcxx.dev 6,945 r /nix/store/bpyj2sk6mh9vlli539jsr0w45rpig5pc-libcxx-6.0.1-dev/include/c++/v1/cxxabi.h llvmPackages_5.libcxx.dev 6,945 r /nix/store/fciabccmn17ijmfxxn50diin0dcjz7m6-libcxx-5.0.2-dev/include/c++/v1/cxxabi.h llvmPackages_8.libcxx.dev 6,987 r /nix/store/8sg9d3j38i3p13sg7pp4bc44szqakj7s-libcxx-8.0.1-dev/include/c++/v1/cxxabi.h llvmPackages_9.libcxx.dev 7,023 r /nix/store/1b5yn18w09n9r50i9162zvn3zq3jn98k-libcxx-9.0.1-dev/include/c++/v1/cxxabi.h llvmPackages_16.libcxxabi.dev 7,237 r /nix/store/02wpjmp2zjjxz13z7g599mniwi25zkcy-libcxxabi-16.0.6-dev/include/c++/v1/cxxabi.h llvmPackages_16.libcxxabi.dev 7,237 r /nix/store/02wpjmp2zjjxz13z7g599mniwi25zkcy-libcxxabi-16.0.6-dev/include/cxxabi.h llvmPackages_10.libcxxabi.dev 7,029 r /nix/store/pwx93kc21scj2ikif98d9zzyzaicag8x-libcxxabi-10.0.1-dev/include/cxxabi.h llvmPackages_7.libcxxabi.dev 6,945 r /nix/store/wx2xdg98ddhrs5iif85w8dgmskgkmjv2-libcxxabi-7.1.0-dev/include/cxxabi.h llvmPackages_9.libcxxabi.dev 7,023 r /nix/store/lhjbqxfqqckm3x2ci9v7vgblmw6jr2jc-libcxxabi-9.0.1-dev/include/cxxabi.h llvmPackages_6.libcxxabi.dev 6,945 r /nix/store/lc41k1nfaslcp8gbgl948ksx52n19j20-libcxxabi-6.0.1-dev/include/cxxabi.h llvmPackages_8.libcxxabi.dev 6,987 r /nix/store/khf9r8s7llkmpsxza19pqavr26j3w6lb-libcxxabi-8.0.1-dev/include/cxxabi.h llvmPackages_10.libcxx.dev 7,029 r /nix/store/j41yp16y29b7gw2bd5c1ljfyyykgsh12-libcxx-10.0.1-dev/include/c++/v1/cxxabi.h llvmPackages_12.libcxx.dev 7,215 r /nix/store/fyr084bq8496099nllfjzapis7mxpj39-libcxx-12.0.1-dev/include/c++/v1/cxxabi.h

because we were copying it manually to libcxx.dev/include

Artturin · 2023-09-17T22:25:13Z

I have to consider whether we should do this or

install -m 644 ../../${pname}/include/__cxxabi_config.h "$dev/include" in libcxxabi postInstall, were already copying cxxabi.h in there

basically if we do this then we don't have to special case cxxabi in the wrapper and can benefit from the hook

nixpkgs/pkgs/build-support/cc-wrapper/setup-hook.sh

Line 71 in fdebb75

if [ -d "$1/include" ]; then

Artturin · 2023-09-17T22:49:28Z

Let's do this

TODO remove now should be unnecessary postInstalls from all libcxxabi's

Artturin · 2023-09-17T22:51:16Z

@natto1784 do you want to re-sign the first commit before merge.

CC @pwaller

Artturin

This restores llvm <15 behaviour

natto1784 · 2023-09-18T01:12:51Z

Let's do this

TODO remove now should be unnecessary postInstalls from all libcxxabi's

Assuming this is redundant

 install -m 644 ../../${pname}/include/${if stdenv.isDarwin then "*" else "cxxabi.h"} "$dev/include"

I am curious as to why is everything installed from the $src itself for darwin but not other platforms

@natto1784 do you want to re-sign the first commit before merge.

CC @pwaller

Oh yeah sure

Removed workaround from llvm 16. Fixes including cxxabi.h on llvm >=15 libcxxStdenv. ```c int main() {} ``` ``` /nix/store/qwnvng0cbyx0bijm654jpmpl0516hfhx-libcxxabi-15.0.7-dev/include/cxxabi.h:20:10: fatal error: '__cxxabi_config.h' file not found ``` Before llvm 15 this used to work because `libcxx` copied the headers from `cxxabi` to it's own `include`, which was then picked up by the line above this one Alternative fix would be to copy all files from `${cxxabi.dev}/include/c++/v1` to `${cxxabi.dev}/include` so the cc-wrapper setup hook would pick them up, but that would depend on in cxxabi being in buildInputs. Signed-off-by: Amneesh Singh <natto@weirdnatto.in>

`#include <cxxabi.h>` `/nix/store/02wpjmp2zjjxz13z7g599mniwi25zkcy-libcxxabi-16.0.6-dev/include/cxxabi.h:20:10: fatal error: '__cxxabi_config.h' file not found`

Artturin · 2023-09-18T01:26:36Z

Let's do this
TODO remove now should be unnecessary postInstalls from all libcxxabi's

Assuming this is redundant
 install -m 644 ../../${pname}/include/${if stdenv.isDarwin then "*" else "cxxabi.h"} "$dev/include"
I am curious as to why is everything installed from the $src itself for darwin but not other platforms

I'm going to leave that for later because removing it is not necessary in this PR.

natto1784 requested a review from Ericson2314 as a code owner September 16, 2023 14:30

natto1784 requested a review from a user September 16, 2023 14:30

Artturin force-pushed the libcxxabi branch from 2498706 to d23b68e Compare September 17, 2023 21:36

Artturin requested review from RaitoBezarius and matthewbauer as code owners September 17, 2023 21:36

Artturin reviewed Sep 17, 2023

View reviewed changes

pkgs/development/compilers/llvm/16/default.nix Outdated

Copy link
Copy Markdown

Member

Artturin Sep 17, 2023

Choose a reason for hiding this comment

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

removed old workaround

Artturin changed the base branch from master to staging September 17, 2023 21:37

Artturin marked this pull request as draft September 17, 2023 21:39

ofborg bot added 10.rebuild-linux: 101-500 This PR causes between 101 and 500 packages to rebuild on Linux. and removed 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. labels Sep 17, 2023

Artturin reviewed Sep 17, 2023

View reviewed changes

Artturin force-pushed the libcxxabi branch from d23b68e to 7476004 Compare September 17, 2023 22:48

Artturin marked this pull request as ready for review September 17, 2023 22:50

Artturin approved these changes Sep 17, 2023

View reviewed changes

delroth added the 12.approvals: 1 This PR was reviewed and approved by one person. label Sep 17, 2023

natto1784 and others added 2 commits September 18, 2023 06:43

tests.cc-wrapper.supported: add test for cxxabi header

42f3292

`#include <cxxabi.h>` `/nix/store/02wpjmp2zjjxz13z7g599mniwi25zkcy-libcxxabi-16.0.6-dev/include/cxxabi.h:20:10: fatal error: '__cxxabi_config.h' file not found`

natto1784 force-pushed the libcxxabi branch from 7476004 to 42f3292 Compare September 18, 2023 01:13

delroth removed the 12.approvals: 1 This PR was reviewed and approved by one person. label Sep 18, 2023

Artturin merged commit 6f27ba8 into NixOS:staging Sep 20, 2023

This was referenced Sep 21, 2023

llvmPackages_15.libcxx: specify LIBCXX_CXX_ABI_{LIBRARY_PATH,INCLUDE_PATHS} for all cxxabis #216273

Closed

llvmPackages_{15}.libcxxStdenv: fix '__cxxabi_config.h' file not found #255756

Closed

wegank mentioned this pull request Dec 7, 2023

llvmPackages_17: init #258614

Merged

12 tasks

rrbutani added the 6.topic: llvm/clang Issues related to llvmPackages, clangStdenv and related label May 27, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

cc-wrapper: add libcxxabi include flag for LLVM#255488

cc-wrapper: add libcxxabi include flag for LLVM#255488
Artturin merged 2 commits intoNixOS:stagingfrom
natto1784:libcxxabi

natto1784 commented Sep 16, 2023

Uh oh!

Artturin Sep 17, 2023

Uh oh!

Artturin Sep 17, 2023

Uh oh!

Artturin commented Sep 17, 2023

Uh oh!

Artturin Sep 17, 2023 •

edited

Loading

Uh oh!

Artturin commented Sep 17, 2023

Uh oh!

Artturin commented Sep 17, 2023

Uh oh!

Artturin commented Sep 17, 2023 •

edited

Loading

Uh oh!

Artturin left a comment

Uh oh!

natto1784 commented Sep 18, 2023

Uh oh!

Artturin commented Sep 18, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

natto1784 commented Sep 16, 2023

Description of changes

Things done

Uh oh!

Artturin Sep 17, 2023

Choose a reason for hiding this comment

Uh oh!

Artturin Sep 17, 2023

Choose a reason for hiding this comment

Uh oh!

Artturin commented Sep 17, 2023

Uh oh!

Artturin Sep 17, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Artturin commented Sep 17, 2023

Uh oh!

Artturin commented Sep 17, 2023

Uh oh!

Artturin commented Sep 17, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Artturin left a comment

Choose a reason for hiding this comment

Uh oh!

natto1784 commented Sep 18, 2023

Uh oh!

Artturin commented Sep 18, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Artturin Sep 17, 2023 •

edited

Loading

Artturin commented Sep 17, 2023 •

edited

Loading