Skip to content

default-gcc-version: 12 -> 13 if stdenv.isAarch64#272674

Merged
vcunat merged 2 commits intoNixOS:stagingfrom
wegank:default-gcc-version-aarch64-linux
Jan 16, 2024
Merged

default-gcc-version: 12 -> 13 if stdenv.isAarch64#272674
vcunat merged 2 commits intoNixOS:stagingfrom
wegank:default-gcc-version-aarch64-linux

Conversation

@wegank
Copy link
Member

@wegank wegank commented Dec 7, 2023

Description of changes

default-gcc-version was bumped in #268097 on x86_64-linux, on i686-linux, on armv7l-linux, on powerpc64le-linux, on riscv64-linux...

But not on aarch64-linux??

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/)
  • 24.05 Release Notes (or backporting 23.05 and 23.11 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.

Add a 👍 reaction to pull requests you find important.

@wegank
Copy link
Member Author

wegank commented Dec 7, 2023

@ofborg build stdenv stdenv.passthru.tests

@wegank
Copy link
Member Author

wegank commented Dec 7, 2023

@ofborg build python3Packages.fasttext python3Packages.fasttext.passthru.tests

@wegank
Copy link
Member Author

wegank commented Dec 7, 2023

@ofborg build libhwy libhwy.passthru.tests

@ofborg ofborg bot added the 8.has: clean-up This PR removes packages or removes other cruft label Dec 7, 2023
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Dec 7, 2023
@wegank
Copy link
Member Author

wegank commented Dec 7, 2023

$ nix-build -A libhwy
...
[66/137] Building CXX object CMakeFiles/shuffle4_test.dir/hwy/tests/shuffle4_test.cc.o
FAILED: CMakeFiles/shuffle4_test.dir/hwy/tests/shuffle4_test.cc.o 
/nix/store/vlyjcld1smalc4sgalddw5j73xgd5kn7-gcc-wrapper-13.2.0/bin/g++ -DHWY_STATIC_DEFINE -I/build/source -isystem /nix/store/4pcmwfklwmsh0sxcgvdlmyg8xcy1zm96-gtest-1.12.1-dev/include -O3 -DNDEBUG -fPIE -fvisibility=hidden -fvisibility-inlines-hidden -Wno-builtin-macro-redefined -D__DATE__=\"redacted\" -D__TIMESTAMP__=\"redacted\" -D__TIME__=\"redacted\" -fmerge-all-constants -Wall -Wextra -Wconversion -Wsign-conversion -Wvla -Wnon-virtual-dtor -fmath-errno -fno-exceptions -DHWY_IS_TEST=1 -MD -MT CMakeFiles/shuffle4_test.dir/hwy/tests/shuffle4_test.cc.o -MF CMakeFiles/shuffle4_test.dir/hwy/tests/shuffle4_test.cc.o.d -o CMakeFiles/shuffle4_test.dir/hwy/tests/shuffle4_test.cc.o -c /build/source/hwy/tests/shuffle4_test.cc
during GIMPLE pass: fre
/build/source/hwy/tests/shuffle4_test.cc: In static member function 'static void hwy::N_SVE::TestPer4LaneBlockShuffle::TestTblLookupPer4LaneBlkShuf(D, size_t, hwy::N_SVE::TFromD<D>*, hwy::N_SVE::TFromD<D>*) [with D = hwy::N_SVE::Simd<long int, 32, 0>]':
/build/source/hwy/tests/shuffle4_test.cc:222:1: internal compiler error: in const_unop, at fold-const.cc:1884
  222 | }  // namespace hwy
      | ^
0x1a0cec7 diagnostic_impl(rich_location*, diagnostic_metadata const*, int, char const*, std::__va_list*, diagnostic_t)
        ???:0
0x1a0dafb internal_error(char const*, ...)
        ???:0
0x708d37 fancy_abort(char const*, int, char const*)
        ???:0
0xb1bea7 const_unop(tree_code, tree_node*, tree_node*)
        ???:0
0x1233aeb gimple_resimplify1(gimple**, gimple_match_op*, tree_node* (*)(tree_node*))
        ???:0
0x126bc97 gimple_simplify(gimple*, gimple_match_op*, gimple**, tree_node* (*)(tree_node*), tree_node* (*)(tree_node*))
        ???:0
0xb5a71b gimple_fold_stmt_to_constant_1(gimple*, tree_node* (*)(tree_node*), tree_node* (*)(tree_node*))
        ???:0
0xfbe15b visit_stmt(gimple*, bool) [clone .isra.0]
        ???:0
0xfbeccf process_bb(rpo_elim&, basic_block_def*, bool, bool, bool, bool, bool, bitmap_head*, bool)
        ???:0
0xfc0a87 do_rpo_vn_1(function*, edge_def*, bitmap_head*, bool, bool, vn_lookup_kind)
        ???:0
0xfc160b (anonymous namespace)::pass_fre::execute(function*)
        ???:0
Please submit a full bug report, with preprocessed source (by using -freport-bug).
Please include the complete backtrace with any bug report.
See <https://gcc.gnu.org/bugs/> for instructions.
[67/137] Building CXX object CMakeFiles/compare_test.dir/hwy/tests/compare_test.cc.o
FAILED: CMakeFiles/compare_test.dir/hwy/tests/compare_test.cc.o 

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110280 (fixed in GCC 13.2.1, 14.0)

@wegank wegank marked this pull request as draft December 7, 2023 16:13
@ofborg ofborg bot added the 10.rebuild-linux-stdenv This PR causes stdenv to rebuild on Linux and must target a staging branch. label Dec 7, 2023
@ofborg ofborg bot requested a review from SuperSandro2000 December 7, 2023 22:46
@ofborg ofborg bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. and removed 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Dec 7, 2023
@ghost
Copy link

ghost commented Dec 7, 2023

But not on aarch64-linux??

Correct; I don't have enough aarch64-linux builders to test that. It is a Hydra-built platform, so I don't want to cause unknown amounts of breakage there.

For the non-Hydra platforms (powerpc64le, mips*, etc), well, those of us who run those platforms sort of expect breakage from version bumps. Goes with the territory. FWIW I fixed all the breakage on powerpc64le and mips64el for the packages I use (a lot on powerpc64le, but for mips64el just the stuff my routers run) and I have been running a gcc13-built system on those machines for two weeks now.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

I am in favor of merging this to staging after 268097 makes it to master.

@ghost
Copy link

ghost commented Dec 7, 2023

After both #268097 and this PR make to master we can start pestering @reckenrode to see about doing Darwin as well.

@ghost

This comment was marked as outdated.

@ghost
Copy link

ghost commented Dec 7, 2023

Ah, never mind, I see you cherry-picked the fix.

@wegank wegank force-pushed the default-gcc-version-aarch64-linux branch from 3ac13a0 to b384f04 Compare January 12, 2024 19:54
@wegank wegank marked this pull request as ready for review January 12, 2024 19:55
@yu-re-ka
Copy link
Contributor

I am seeing a new failure on pkgs.reptyr: https://termbin.com/nqqk

Copy link
Contributor

@yu-re-ka yu-re-ka left a comment

Choose a reason for hiding this comment

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

Besides reptyr, I could not find any issues. Built my laptop with master + this applied

@vcunat vcunat changed the title default-gcc-version: 12 -> 13 if !stdenv.isDarwin default-gcc-version: 12 -> 13 if stdenv.isAarch64 Jan 16, 2024
@vcunat vcunat merged commit fcc077d into NixOS:staging Jan 16, 2024
@wegank wegank deleted the default-gcc-version-aarch64-linux branch January 16, 2024 11:28
@ghost ghost mentioned this pull request Jan 18, 2024
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: clean-up This PR removes packages or removes other cruft 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. 10.rebuild-linux-stdenv This PR causes stdenv to rebuild on Linux and must target a staging branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants