Skip to content

openmp: fix cross compile#197674

Merged
Artturin merged 1 commit intoNixOS:masterfrom
squalus:openmp-cross
Nov 22, 2022
Merged

openmp: fix cross compile#197674
Artturin merged 1 commit intoNixOS:masterfrom
squalus:openmp-cross

Conversation

@squalus
Copy link
Member

@squalus squalus commented Oct 25, 2022

Description of changes

Fixes cross compile for openmp.

Doesn't cause any rebuilds.

Tested cross builds from x86_64-linux -> aarch64-linux on llvmPackages 10 through 14. Tested all evaluations. Tested compilation of a few dependent packages.

The following attrs are now expected to build:

pkgsCross.aarch64-multiplatform.llvmPackages_5.openmp
pkgsCross.aarch64-multiplatform.llvmPackages_6.openmp
pkgsCross.aarch64-multiplatform.llvmPackages_7.openmp
pkgsCross.aarch64-multiplatform.llvmPackages_8.openmp
pkgsCross.aarch64-multiplatform.llvmPackages_9.openmp
pkgsCross.aarch64-multiplatform.llvmPackages_10.openmp
pkgsCross.aarch64-multiplatform.llvmPackages_11.openmp
pkgsCross.aarch64-multiplatform.llvmPackages_12.openmp
pkgsCross.aarch64-multiplatform.llvmPackages_13.openmp
pkgsCross.aarch64-multiplatform.llvmPackages_14.openmp

The following attrs are expected to produce the same output as the normal native build:

pkgsCross.aarch64-multiplatform.buildPackages.llvmPackages_5.openmp
pkgsCross.aarch64-multiplatform.buildPackages.llvmPackages_6.openmp
pkgsCross.aarch64-multiplatform.buildPackages.llvmPackages_7.openmp
pkgsCross.aarch64-multiplatform.buildPackages.llvmPackages_8.openmp
pkgsCross.aarch64-multiplatform.buildPackages.llvmPackages_9.openmp
pkgsCross.aarch64-multiplatform.buildPackages.llvmPackages_10.openmp
pkgsCross.aarch64-multiplatform.buildPackages.llvmPackages_11.openmp
pkgsCross.aarch64-multiplatform.buildPackages.llvmPackages_12.openmp
pkgsCross.aarch64-multiplatform.buildPackages.llvmPackages_13.openmp
pkgsCross.aarch64-multiplatform.buildPackages.llvmPackages_14.openmp
Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • 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/)
  • 22.11 Release Notes (or backporting 22.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
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@squalus squalus requested a review from matthewbauer as a code owner October 25, 2022 06:52
@squalus squalus added the 6.topic: cross-compilation Building packages on a different platform than they will be used on label Oct 25, 2022
@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 Oct 25, 2022
delroth
delroth reviewed Oct 31, 2022
View reviewed changes
pkgs/development/compilers/llvm/10/openmp/default.nix
nativeBuildInputs = [ cmake perl ];
buildInputs = [ llvm ];
buildInputs = [
(if stdenv.buildPlatform == stdenv.hostPlatform then llvm else targetLlvm)
Copy link
Contributor

@delroth delroth Oct 31, 2022

Choose a reason for hiding this comment

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

(Commenting only on this one particular instance, but obviously applies to all.)

When buildPlatform == hostPlatform is there a meaningful difference between llvm and targetLlvm? In other words, could we always use targetLlvm here?

If this is done only to avoid rebuilds, I'd personally prefer rebuilds to long term code complexity / "technical debt".

Copy link
Member Author

@squalus squalus Oct 31, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

Here's why: if I simply use targetLlvm, then pkgsCross.aarch64-multiplatform.buildPackages.llvmPackages_14.openmp will result in a failed build. The "build" version of the llvm libraries is included instead of the expected "host" version. I admit I'm not sure why this is the case. There may be a better way to model this.

@rrbutani rrbutani mentioned this pull request Nov 7, 2022
Merged
92 tasks
@Mic92 Mic92 requested a review from Ericson2314 November 20, 2022 19:25
Artturin
Artturin approved these changes Nov 21, 2022
View reviewed changes
Copy link
Member

@Artturin Artturin left a comment

Choose a reason for hiding this comment

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

eh good enough i think. something should be done about the code duplication going on with llvmPackages

@Artturin
Copy link
Member

Artturin commented Nov 21, 2022

@ofborg build pkgsCross.aarch64-multiplatform.llvmPackages_14.openmp

@Artturin Artturin merged commit 006c56a into NixOS:master Nov 22, 2022
rrbutani added a commit to rrbutani/nixpkgs that referenced this pull request Jan 8, 2023
@rrbutani
llvmPackages_15: apply some patches from llvmPackages_14
8c4b8b5 
See NixOS#194634 for details.

PRs:
  - NixOS#191372
  - NixOS#190936
  - NixOS#82131
  - NixOS#199844
  - NixOS#197674
  - NixOS#184408
  - NixOS#193004
ElvishJerricco pushed a commit to ElvishJerricco/nixpkgs that referenced this pull request Jan 8, 2023
@rrbutani @ElvishJerricco
llvmPackages_15: apply some patches from llvmPackages_14
f3377f0 
See NixOS#194634 for details.

PRs:
  - NixOS#191372
  - NixOS#190936
  - NixOS#82131
  - NixOS#199844
  - NixOS#197674
  - NixOS#184408
  - NixOS#193004
rrbutani added a commit to rrbutani/nixpkgs that referenced this pull request Jan 10, 2023
@rrbutani
llvmPackages_15: apply some patches from llvmPackages_14
589713c 
See NixOS#194634 for details.

PRs:
  - NixOS#191372
  - NixOS#190936
  - NixOS#82131
  - NixOS#199844
  - NixOS#197674
  - NixOS#184408
  - NixOS#193004
rrbutani added a commit to rrbutani/nixpkgs that referenced this pull request Jan 27, 2023
@rrbutani
llvmPackages_15: apply some patches from llvmPackages_14
81ef82a 
See NixOS#194634 for details.

PRs:
  - NixOS#191372
  - NixOS#190936
  - NixOS#82131
  - NixOS#199844
  - NixOS#197674
  - NixOS#184408
  - NixOS#193004
github-actions bot pushed a commit that referenced this pull request Jan 28, 2023
@rrbutani @github-actions
llvmPackages_15: apply some patches from llvmPackages_14
4fb7083 
See #194634 for details.

PRs:
  - #191372
  - #190936
  - #82131
  - #199844
  - #197674
  - #184408
  - #193004
(cherry picked from commit 81ef82a)
K900 pushed a commit that referenced this pull request Jan 28, 2023
@rrbutani @K900
llvmPackages_15: apply some patches from llvmPackages_14
87e4857 
See #194634 for details.

PRs:
  - #191372
  - #190936
  - #82131
  - #199844
  - #197674
  - #184408
  - #193004
xanderio pushed a commit to xanderio/nixpkgs that referenced this pull request Feb 13, 2023
@rrbutani @xanderio
llvmPackages_15: apply some patches from llvmPackages_14
871ab8d 
See NixOS#194634 for details.

PRs:
  - NixOS#191372
  - NixOS#190936
  - NixOS#82131
  - NixOS#199844
  - NixOS#197674
  - NixOS#184408
  - NixOS#193004
@rrbutani rrbutani mentioned this pull request Mar 26, 2023
Closed
48 tasks
rrbutani added a commit to rrbutani/nixpkgs that referenced this pull request Apr 3, 2023
@rrbutani
llvmPackages_git.openmp: apply NixOS#197674 (fix cross compile)
61b348f 
this was backported to `llvmPackages_15` in
81ef82a
rrbutani added a commit to rrbutani/nixpkgs that referenced this pull request Apr 5, 2023
@rrbutani
llvmPackages_git.openmp: apply NixOS#197674 (fix cross compile)
235588c 
this was backported to `llvmPackages_15` in
81ef82a
RaitoBezarius pushed a commit that referenced this pull request Apr 14, 2023
@rrbutani @RaitoBezarius
llvmPackages_git.openmp: apply #197674 (fix cross compile)
b027a36 
this was backported to `llvmPackages_15` in
81ef82a
RaitoBezarius pushed a commit to rrbutani/nixpkgs that referenced this pull request Apr 16, 2023
@rrbutani @RaitoBezarius
llvmPackages_git.openmp: apply NixOS#197674 (fix cross compile)
f49825c 
this was backported to `llvmPackages_15` in
81ef82a
RaitoBezarius added a commit that referenced this pull request Apr 19, 2023
@RaitoBezarius
Merge pull request #222903 from rrbutani/fix/llvm-git-openmp-cross
df7e676 
llvmPackages_git.openmp: apply #197674 (fix cross compile)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Artturin Artturin Artturin approved these changes

@matthewbauer matthewbauer Awaiting requested review from matthewbauer

@Ericson2314 Ericson2314 Awaiting requested review from Ericson2314

+1 more reviewer

@delroth delroth delroth left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

6.topic: cross-compilation Building packages on a different platform than they will be used on 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.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@squalus @Artturin @delroth