Skip to content

Commit

Permalink
mesa_noglu: fix build
Browse files Browse the repository at this point in the history
LLVM’s `LLVM_TARGETS_TO_BUILD` build flag defauls to `all`, which contains
`AMDGPU` among others. [1] Changes in llvm [2] switched to explicitly listing
host and target platforms, excluding the AMDGPU target, which is required
for Mesa to build.

[1]: https://github.com/llvm-mirror/llvm/blob/db50b6fe399b8ad8caef80fd8ee77607fb051fa5/CMakeLists.txt#L286-L302
[2]: #52031
  • Loading branch information
jtojnar committed Dec 24, 2018
1 parent 192f417 commit ff22d90
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion pkgs/development/compilers/llvm/6/llvm.nix
Expand Up @@ -13,7 +13,8 @@
, fetchpatch
, debugVersion ? false
, enableManpages ? false
, enableTargets ? [ stdenv.hostPlatform stdenv.targetPlatform ]
# Mesa requires AMDGPU target
, enableTargets ? [ stdenv.hostPlatform stdenv.targetPlatform "AMDGPU" ]
, enableSharedLibraries ? true
}:

Expand Down
3 changes: 2 additions & 1 deletion pkgs/development/compilers/llvm/7/llvm.nix
Expand Up @@ -14,7 +14,8 @@
, debugVersion ? false
, enableManpages ? false
, enableSharedLibraries ? true
, enableTargets ? [ stdenv.hostPlatform stdenv.targetPlatform ]
# Mesa requires AMDGPU target
, enableTargets ? [ stdenv.hostPlatform stdenv.targetPlatform "AMDGPU" ]
, enablePFM ? !stdenv.isDarwin
}:

Expand Down

0 comments on commit ff22d90

Please sign in to comment.