[PATCH] D106860: [clang][AArch64][SVE] Avoid going through memory for fixed/scalable predicate casts

Bradley Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 29 08:44:06 PDT 2021


bsmith added inline comments.


================
Comment at: clang/lib/CodeGen/CGExprScalar.cpp:2102
+          Src = Builder.CreateBitCast(Src, SrcTy);
+        }
         if (ScalableSrc->getElementType() == FixedDst->getElementType()) {
----------------
junparser wrote:
> I think this may also works for casting between vectors with different element types.
A similar argument applies here as the other related ticket, in principal we could, however it's not clear that there is a good use case for writing code that would make use of this. So for now it's probably best to just deal with predicates which are definitely a problem and other cases as they arise.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106860



More information about the cfe-commits mailing list