[PATCH] D138788: [SVE] Change some bfloat lane intrinsics to use i32 immediates

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added inline comments.


================
Comment at: llvm/include/llvm/IR/IntrinsicsAArch64.td:2527
 
-def int_aarch64_sve_bfdot_lane   : SVE_4Vec_BF16_Indexed;
-def int_aarch64_sve_bfmlalb_lane : SVE_4Vec_BF16_Indexed;
-def int_aarch64_sve_bfmlalt_lane : SVE_4Vec_BF16_Indexed;
+def int_aarch64_sve_bfdot_lane       : SVE_4Vec_BF16_Indexed;
+def int_aarch64_sve_bfdot_lane_i32   : SVE_4Vec_BF16_Indexed_I32;
----------------
david-arm wrote:
> sdesmalen wrote:
> > do you also want to remove the old intrinsics?
> Good spot! Turns out that not only had I done this wrong, but I'd also missed out upgrades for bfmlalb/t too. :)
Having `_i32` in the name is confusing because it'll come out as `.i32` when printed in IR, which looks like a type suffix but in this case it's actually part of the name.

With that said, do you have to change the name?  That seems unfortunate given this is a bug fix.

If it's absolutely necessary then I suggestion using `_v2` to signify this is the second version of this intrinsic.


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

https://reviews.llvm.org/D138788