[llvm] acd64d8 - [Support/BLAKE3] CMake: Check for `IS_X64` or `CMAKE_OSX_ARCHITECTURES` before adding the assembly files
Argyrios Kyrtzidis via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 22 10:52:55 PDT 2022
Sorry about that, I had made a copy-paste mistake in my changes, I fixed it here: https://github.com/llvm/llvm-project/commit/69d9437472851290b5db54661c336284f7ffc9a8
> On Jun 22, 2022, at 10:39 AM, Mehdi AMINI <joker.eph at gmail.com> wrote:
>
> Hi Argyrios,
>
> I see some failure right now with clang-5: https://buildkite.com/mlir/mlir-core/builds/23452#01818c5c-a30d-4172-92d6-44d6ead085a7 <https://buildkite.com/mlir/mlir-core/builds/23452#01818c5c-a30d-4172-92d6-44d6ead085a7> ; can you look into this?
>
> Thanks,
>
> --
> Mehdi
>
>
> On Tue, Jun 21, 2022 at 6:48 PM Argyrios Kyrtzidis via llvm-commits <llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>> wrote:
>
> Author: Argyrios Kyrtzidis
> Date: 2022-06-21T09:48:07-07:00
> New Revision: acd64d8e85f69467c64af833ef6b38b807bd9b4b
>
> URL: https://github.com/llvm/llvm-project/commit/acd64d8e85f69467c64af833ef6b38b807bd9b4b <https://github.com/llvm/llvm-project/commit/acd64d8e85f69467c64af833ef6b38b807bd9b4b>
> DIFF: https://github.com/llvm/llvm-project/commit/acd64d8e85f69467c64af833ef6b38b807bd9b4b.diff <https://github.com/llvm/llvm-project/commit/acd64d8e85f69467c64af833ef6b38b807bd9b4b.diff>
>
> LOG: [Support/BLAKE3] CMake: Check for `IS_X64` or `CMAKE_OSX_ARCHITECTURES` before adding the assembly files
>
> This should fix `clang-ppc64-aix` builder (https://lab.llvm.org/buildbot/#/builders/214 <https://lab.llvm.org/buildbot/#/builders/214>)
>
> Added:
>
>
> Modified:
> llvm/lib/Support/BLAKE3/CMakeLists.txt
>
> Removed:
>
>
>
> ################################################################################
> diff --git a/llvm/lib/Support/BLAKE3/CMakeLists.txt b/llvm/lib/Support/BLAKE3/CMakeLists.txt
> index df1805918587..4fd60a99bbfb 100644
> --- a/llvm/lib/Support/BLAKE3/CMakeLists.txt
> +++ b/llvm/lib/Support/BLAKE3/CMakeLists.txt
> @@ -53,12 +53,8 @@ if (CAN_USE_ASSEMBLER)
> disable_blake3_x86_simd()
> endif()
> else()
> - check_symbol_exists(__i386__ "" IS_X32)
> - if (IS_X32)
> - # blake3 C code autoenables SIMD for i386, but those implementations are
> - # only available via the intrinsics sources which we don't enable here.
> - disable_blake3_x86_simd()
> - else()
> + check_symbol_exists(__x86_64__ "" IS_X64)
> + if (IS_X64 OR CMAKE_OSX_ARCHITECTURES)
> # In a macOS Universal build (setting CMAKE_OSX_ARCHITECTURES to multiple
> # values), compilation of the source files will target multiple architectures
> # (each source file is internally compiled once for each architecture).
> @@ -70,13 +66,11 @@ if (CAN_USE_ASSEMBLER)
> blake3_avx2_x86-64_unix.S
> blake3_avx512_x86-64_unix.S
> )
> - check_c_compiler_flag("-mavx512vl" SUPPORTS_avx512vl)
> - if (SUPPORTS_avx512vl)
> - # Clang-6 needs this flag. We also suppress '-Wunused-command-line-argument'
> - # in case the file is included with arm architecture.
> - set_source_files_properties(blake3_avx512_x86-64_unix.S
> - PROPERTIES COMPILE_OPTIONS "-mavx512vl;-Wno-unused-command-line-argument")
> - endif()
> + # Clang-6 needs this flag.
> + set_source_files_properties(blake3_avx512_x86-64_windows_gnu.S
> + PROPERTIES COMPILE_OPTIONS "-mavx512vl")
> + else()
> + disable_blake3_x86_simd()
> endif()
> endif()
> else()
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits <https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220622/ff2f3662/attachment.html>
More information about the llvm-commits
mailing list