[llvm] [AArch64][SVE] Instcombine uzp1/reinterpret svbool to use vector.insert (PR #81069)

Paul Walker via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 14 05:36:28 PST 2024


================
@@ -1630,6 +1630,36 @@ static std::optional<Instruction *> instCombineSVETBL(InstCombiner &IC,
   return IC.replaceInstUsesWith(II, VectorSplat);
 }
 
+static std::optional<Instruction *> instCombineSVEUzp1(InstCombiner &IC,
+                                                       IntrinsicInst &II) {
+  Value *A, *B;
+  Type *RetTy = II.getType();
+  constexpr Intrinsic::ID From_SV = Intrinsic::aarch64_sve_convert_from_svbool;
+  constexpr Intrinsic::ID To_SV = Intrinsic::aarch64_sve_convert_to_svbool;
----------------
paulwalker-arm wrote:

Not sure the coding standard is keen on `_` in variable names.  Also I read `SV` as "Scalable Vector", which doesn't mean much in the context.  What About `FromSVB` to solve both issues?

https://github.com/llvm/llvm-project/pull/81069


More information about the llvm-commits mailing list