[PATCH] D94765: [WIP]Expand masked mem intrinsics correctly wrt big-endian

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 16 09:00:27 PST 2021


dmgreen added a comment.

Thanks for the patch. There is a patch to make MVE consistent with the rest of MVE in D94867 <https://reviews.llvm.org/D94867>. This will need rebasing on top of that, with update tests to make the two consistent again.



================
Comment at: llvm/lib/Transforms/Scalar/ScalarizeMaskedMemIntrin.cpp:321-322
 
   // If the mask is not v1i1, use scalar bit test operations. This generates
   // better results on X86 at least.
   Value *SclrMask;
----------------
It's a shame we can't just use extract elements - that would simplify this a lot!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94765



More information about the llvm-commits mailing list