Skip to content
Permalink

Comparing changes

This is a direct comparison between two commits made in this repository or its related repositories. View the default comparison for this range or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: JuliaPackaging/Yggdrasil
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 57d0821d538a6f6bfb3bde701a4b8565d85e9ca1
Choose a base ref
..
head repository: JuliaPackaging/Yggdrasil
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: cf0bf00cbcd0b1dfcadfb6e904465b4f2980b9ff
Choose a head ref
Showing with 2 additions and 8 deletions.
  1. +2 −8 O/oneDNN/build_tarballs.jl
10 changes: 2 additions & 8 deletions O/oneDNN/build_tarballs.jl
Original file line number Diff line number Diff line change
@@ -16,18 +16,12 @@ script = raw"""
cd $WORKSPACE/srcdir/oneDNN
mkdir build && cd build/
if [[ $target == aarch64* ]]; then
cmake_extra_args="-DCMAKE_SYSTEM_PROCESSOR=AARCH64"
fi
cmake \
-DCMAKE_INSTALL_PREFIX=${prefix} \
-DCMAKE_TOOLCHAIN_FILE=${CMAKE_TARGET_TOOLCHAIN} \
-DCMAKE_BUILD_TYPE=Release \
-DONEDNN_BUILD_EXAMPLES=OFF \
-DONEDNN_BUILD_TESTS=OFF \
-DONEDNN_CPU_RUNTIME=TBB \
-DTBBROOT=$prefix \
$cmake_extra_args \
..
make -j${nproc}
make install
@@ -37,6 +31,7 @@ make install
# platforms are passed in on the command line
platforms = supported_platforms()
filter!(p -> nbits(p) == 64, platforms) # oneDNN supports 64 bit platforms only
filter!(p -> arch(p) == "x86_64", platforms) # Temp. disabled non-x86_64
filter!(p -> libc(p) != "musl", platforms) # musl fails to link with ssp(?)
filter!(p -> os(p) != "windows", platforms) # windows fails to compile: error: ‘_MCW_DN’ was not declared in this scope
platforms = expand_cxxstring_abis(platforms)
@@ -48,8 +43,7 @@ products = [

# Dependencies that must be installed before this package can be built
dependencies = [
Dependency(PackageSpec(name="CompilerSupportLibraries_jll", uuid="e66e0078-7015-5450-92f7-15fbd957f2ae")),
Dependency("oneTBB_jll")
Dependency(PackageSpec(name="CompilerSupportLibraries_jll", uuid="e66e0078-7015-5450-92f7-15fbd957f2ae"))
]

# Build the tarballs, and possibly a `build.jl` as well.