[PATCH] D138760: BLAKE3: do not try to use neon on big-endian aarch64

Duncan P. N. Exon Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 30 11:15:30 PST 2022


dexonsmith added a comment.

In D138760#4020385 <https://reviews.llvm.org/D138760#4020385>, @MaskRay wrote:

> In D138760#4018383 <https://reviews.llvm.org/D138760#4018383>, @he32 wrote:
>
>> Thanks for the hint / redirect.
>> I've submitted a pull request at https://github.com/BLAKE3-team/BLAKE3/pull/280
>
> Thank you! If we take the patch here, we may risk losing the change when someone upgrades BLAKE3. So it's better to ensure that it works in the upstream...

Can’t we have a test/bot for that? Seems awkward to depend on this getting integrated upstream first. This is just unblocking builds so I’d be inclined to take it…


Repository:
  rZORG LLVM Github Zorg

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

https://reviews.llvm.org/D138760



More information about the llvm-commits mailing list