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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
