[clang] [llvm] [X86][AVX10.2-SATCVT][NFC] Remove NE from intrinsic and instruction name (PR #123275)
Phoebe Wang via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 22 06:44:54 PST 2025
================
@@ -603,13 +603,13 @@ defm VCVTPS2IUBS : avx10_sat_cvt_base<0x6b, "vcvtps2iubs", SchedWriteVecIMul,
X86vcvtp2iubsRnd>,
AVX512PDIi8Base, T_MAP5, EVEX_CD8<32, CD8VF>;
-defm VCVTTNEBF162IBS : avx10_sat_cvt_base<0x68, "vcvttnebf162ibs",
- SchedWriteVecIMul, X86vcvttp2ibs,
- avx512vl_i16_info, avx512vl_bf16_info>,
+defm VCVTTBF162IBS : avx10_sat_cvt_base<0x68, "vcvttbf162ibs",
+ SchedWriteVecIMul, X86vcvttp2ibs,
+ avx512vl_i16_info, avx512vl_bf16_info>,
AVX512XDIi8Base, T_MAP5, EVEX_CD8<16, CD8VF>;
-defm VCVTTNEBF162IUBS : avx10_sat_cvt_base<0x6a, "vcvttnebf162iubs",
- SchedWriteVecIMul, X86vcvttp2iubs,
- avx512vl_i16_info, avx512vl_bf16_info>,
+defm VCVTTBF162IUBS : avx10_sat_cvt_base<0x6a, "vcvttbf162iubs",
+ SchedWriteVecIMul, X86vcvttp2iubs,
+ avx512vl_i16_info, avx512vl_bf16_info>,
AVX512XDIi8Base, T_MAP5, EVEX_CD8<16, CD8VF>;
----------------
phoebewang wrote:
There's no uniform rule on this. So I usually just pay attention to arguments in `<>`.
Align `AVX512XDIi8Base` with `avx10_sat_cvt_base` doesn't make much sense, because one is `class` while the other is `multiclass`. We uaually align them if we have more than one `multiclass`.
https://github.com/llvm/llvm-project/pull/123275
More information about the llvm-commits
mailing list