paulwalker-arm accepted this revision.
paulwalker-arm added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/include/llvm/IR/IntrinsicsAArch64.td:1517
-class SVE_4Vec_BF16_Indexed
+class SVE_4Vec_BF16_Indexed_V2
: DefaultAttrsIntrinsic<[llvm_nxv4f32_ty],
----------------
I suppose you don't actually need to change the name of this class.
================
Comment at: llvm/lib/IR/AutoUpgrade.cpp:3977
+ LLVMContext &Ctx = F->getParent()->getContext();
+ SmallVector<Value *, 2> Args(CI->args());
+ Args[3] = ConstantInt::get(Type::getInt32Ty(Ctx),
----------------
This may as well be 4 given we know the exact number of arguments.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138788/new/
https://reviews.llvm.org/D138788
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
