Skip to content

llvmPackages: llvmPackages_{16,18} -> llvmPackages_19#354107

Merged
emilazy merged 73 commits intostagingfrom
llvm-19
Nov 20, 2024
Merged

llvmPackages: llvmPackages_{16,18} -> llvmPackages_19#354107
emilazy merged 73 commits intostagingfrom
llvm-19

Conversation

@emilazy
Copy link
Member

@emilazy emilazy commented Nov 7, 2024

This is the work‐in‐progress PR for the 25.05 bump to LLVM 19. We’ve had a few Darwin people working on this independently, but the issues and patches are piling up and it’d be good to have a common branch to work on together. It’s currently based on my Darwin minimum version bump PR since that will land in the same cycle, but will be retargeted to staging once the branch‐off happens and that PR is merged. Please feel free to send PRs to this branch.

Known blockers:

cc @NixOS/llvm @NixOS/darwin-core

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.11 Release Notes (or backporting 23.11 and 24.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.

Add a 👍 reaction to pull requests you find important.

@emilazy emilazy added 6.topic: darwin Running or building packages on Darwin 6.topic: llvm/clang Issues related to llvmPackages, clangStdenv and related 2.status: wait for branch‐off Waiting for the next Nixpkgs branch‐off labels Nov 7, 2024
@emilazy emilazy marked this pull request as draft November 7, 2024 00:01
@github-actions github-actions bot removed the 6.topic: llvm/clang Issues related to llvmPackages, clangStdenv and related label Nov 7, 2024
@ofborg ofborg bot added 10.rebuild-darwin-stdenv This PR causes stdenv to rebuild on Darwin and must target a staging branch. 8.has: clean-up This PR removes packages or removes other cruft 8.has: package (new) This PR adds a new package labels Nov 7, 2024
@ofborg ofborg bot added 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 5001+ This PR causes many rebuilds on Darwin and must target the staging branches. 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. labels Nov 7, 2024
@github-actions github-actions bot added the 6.topic: rust General-purpose programming language emphasizing performance, type safety, and concurrency. label Nov 7, 2024
@emilazy emilazy force-pushed the emily/push-nkyqpvqvxwzn branch from f17e590 to f1e9c7f Compare November 7, 2024 23:09
@github-actions github-actions bot removed the 6.topic: darwin Running or building packages on Darwin label Nov 7, 2024
@emilazy emilazy mentioned this pull request Nov 8, 2024
13 tasks
@emilazy emilazy force-pushed the emily/push-nkyqpvqvxwzn branch from f1e9c7f to 6e157fb Compare November 8, 2024 18:57
@github-actions github-actions bot added the 6.topic: stdenv Standard environment label Nov 8, 2024
emilazy and others added 13 commits November 20, 2024 02:33
These work on Darwin, and using the system libpng avoids an issue trying to build the vendored version of libpng with clang 18.
python3Packages.pylibjpeg-openjpeg tries to access openjpeg.src which
was changed to srcs in 1f9a1d0. Change
it back.
…hen compiling + linking

`https://github.com/llvm/llvm-project/issues/116278`

make clang report unsupported option errors / warning when compiling +
linking in the same command.

this works
```
$ clang --target=aarch64 -mpopcnt hello.c
clang: error: unsupported option '-mpopcnt' for target aarch64
```

before change clang swallows the error
```
$ clang --target=aarch64 -mpopcnt hello.c -lc
$ echo $?
0
```

after change error is reported:
```
$ clang --target=aarch64 -mpopcnt hello.c -lc
clang: error: unsupported option '-mpopcnt' for target aarch64
```
libc++ 19 no longer supports Clang < 17.

This reverts commit b034e4c.
@github-actions github-actions bot added the 6.topic: nodejs Node.js is a free, open-source, cross-platform JavaScript runtime environment label Nov 20, 2024
@alyssais
Copy link
Member

Ack.

@emilazy emilazy merged commit d1e5e11 into staging Nov 20, 2024
@emilazy emilazy deleted the llvm-19 branch November 20, 2024 14:47
@trofi
Copy link
Contributor

trofi commented Nov 23, 2024

Bisect says that da65c30 openjpeg: fix up tests broke test in staging on x86_64-linux as:

$ nix build --no-link -f. -L pkgsi686Linux.openjpeg
...
openjpeg> 99% tests passed, 3 tests failed out of 1528
openjpeg> Total Test time (real) =  13.90 sec
openjpeg> The following tests FAILED:
openjpeg>       104 - NR-JP2-file3.jp2-compare2base (Failed)
openjpeg>       1188 - NR-DEC-issue414.jp2-110-decode-md5 (Failed)
openjpeg>       1227 - NR-DEC-p1_04.j2k-134-decode-md5 (Failed)

@nixos-discourse
Copy link

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

https://discourse.nixos.org/t/fixing-gcc-14-related-build-failures-in-nixpkgs/61272/3

emilazy added a commit to emilazy/nixpkgs that referenced this pull request Sep 7, 2025
This has been de facto the case for a while now anyway, given
its inseparability from my `stdenv` and Darwin team roles,
especially after <NixOS#354107> and
<NixOS#440273>. Might as well make
sure I don’t miss pings, although I can’t promise I’ll be able
to proactively review stuff that falls outside the scope that matters
to me.
alyssais pushed a commit that referenced this pull request Sep 8, 2025
This has been de facto the case for a while now anyway, given
its inseparability from my `stdenv` and Darwin team roles,
especially after <#354107> and
<#440273>. Might as well make
sure I don’t miss pings, although I can’t promise I’ll be able
to proactively review stuff that falls outside the scope that matters
to me.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: darwin Running or building packages on Darwin 6.topic: haskell General-purpose, statically typed, purely functional programming language 6.topic: llvm/clang Issues related to llvmPackages, clangStdenv and related 6.topic: nodejs Node.js is a free, open-source, cross-platform JavaScript runtime environment 6.topic: qt/kde Object-oriented framework for GUI creation 6.topic: rust General-purpose programming language emphasizing performance, type safety, and concurrency. 6.topic: stdenv Standard environment 8.has: clean-up This PR removes packages or removes other cruft 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 5001+ This PR causes many rebuilds on Darwin and must target the staging branches. 10.rebuild-darwin-stdenv This PR causes stdenv to rebuild on Darwin and must target a staging branch. 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.

9 participants