[PATCH] D126543: [bazel] BLAKE3: Adopt aarch64 and x86_64.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 27 11:50:08 PDT 2022


MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

Looks great!

  % cd utils/bazel
  % bazel build --config=generic_clang @llvm-project//llvm:Support --copt=-w

I can find sse and avx .o files in `bazel-out/k8-fastbuild/bin/external/llvm-project/llvm/libSupport.so-2.params`.



================
Comment at: utils/bazel/llvm-project-overlay/llvm/BUILD.bazel:205
+            "lib/Support/BLAKE3/blake3_avx512_x86-64_unix.S",
+        ],
+    }),
----------------
https://docs.bazel.build/versions/main/configurable-attributes.html#the-default-condition there should be a `//conditions:default` condition for non-aarch64-nonx86_64 CPUs.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126543/new/

https://reviews.llvm.org/D126543



More information about the llvm-commits mailing list