[clang] [llvm] [X86][AVX10.2-BF16] Remove [NE]P from intrinsic and instruction name (PR #123335)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 17 05:31:19 PST 2025
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff 1274bca2ad5befe56d82ef76100e2c294ca57ce2 ef979383a507277247d36b97572f4bf558e84305 --extensions inc,cpp,h,c -- clang/lib/CodeGen/CGBuiltin.cpp clang/lib/Headers/avx10_2_512bf16intrin.h clang/lib/Headers/avx10_2bf16intrin.h clang/lib/Sema/SemaX86.cpp clang/test/CodeGen/X86/avx10_2_512bf16-builtins.c clang/test/CodeGen/X86/avx10_2bf16-builtins.c llvm/lib/Target/X86/AsmParser/X86AsmParser.cpp llvm/lib/Target/X86/MCTargetDesc/X86ATTInstPrinter.cpp llvm/lib/Target/X86/MCTargetDesc/X86InstComments.cpp llvm/lib/Target/X86/MCTargetDesc/X86InstPrinterCommon.cpp llvm/lib/Target/X86/MCTargetDesc/X86IntelInstPrinter.cpp llvm/lib/Target/X86/X86InstrFMA3Info.cpp llvm/lib/Target/X86/X86IntrinsicsInfo.h llvm/test/TableGen/x86-fold-tables.inc
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/llvm/lib/Target/X86/MCTargetDesc/X86ATTInstPrinter.cpp b/llvm/lib/Target/X86/MCTargetDesc/X86ATTInstPrinter.cpp
index cd89b88f46..6b9527502b 100644
--- a/llvm/lib/Target/X86/MCTargetDesc/X86ATTInstPrinter.cpp
+++ b/llvm/lib/Target/X86/MCTargetDesc/X86ATTInstPrinter.cpp
@@ -166,15 +166,24 @@ bool X86ATTInstPrinter::printVecCompareInstr(const MCInst *MI,
case X86::VCMPPHZrmbi: case X86::VCMPPHZrmbik:
case X86::VCMPPHZrrib: case X86::VCMPPHZrribk:
case X86::VCMPSHZrrib_Int: case X86::VCMPSHZrribk_Int:
- case X86::VCMPBF16Z128rmi: case X86::VCMPBF16Z128rri:
- case X86::VCMPBF16Z256rmi: case X86::VCMPBF16Z256rri:
- case X86::VCMPBF16Zrmi: case X86::VCMPBF16Zrri:
- case X86::VCMPBF16Z128rmik: case X86::VCMPBF16Z128rrik:
- case X86::VCMPBF16Z256rmik: case X86::VCMPBF16Z256rrik:
- case X86::VCMPBF16Zrmik: case X86::VCMPBF16Zrrik:
- case X86::VCMPBF16Z128rmbi: case X86::VCMPBF16Z128rmbik:
- case X86::VCMPBF16Z256rmbi: case X86::VCMPBF16Z256rmbik:
- case X86::VCMPBF16Zrmbi: case X86::VCMPBF16Zrmbik:
+ case X86::VCMPBF16Z128rmi:
+ case X86::VCMPBF16Z128rri:
+ case X86::VCMPBF16Z256rmi:
+ case X86::VCMPBF16Z256rri:
+ case X86::VCMPBF16Zrmi:
+ case X86::VCMPBF16Zrri:
+ case X86::VCMPBF16Z128rmik:
+ case X86::VCMPBF16Z128rrik:
+ case X86::VCMPBF16Z256rmik:
+ case X86::VCMPBF16Z256rrik:
+ case X86::VCMPBF16Zrmik:
+ case X86::VCMPBF16Zrrik:
+ case X86::VCMPBF16Z128rmbi:
+ case X86::VCMPBF16Z128rmbik:
+ case X86::VCMPBF16Z256rmbi:
+ case X86::VCMPBF16Z256rmbik:
+ case X86::VCMPBF16Zrmbi:
+ case X86::VCMPBF16Zrmbik:
if (Imm >= 0 && Imm <= 31) {
OS << '\t';
printCMPMnemonic(MI, /*IsVCMP*/true, OS);
diff --git a/llvm/lib/Target/X86/MCTargetDesc/X86InstPrinterCommon.cpp b/llvm/lib/Target/X86/MCTargetDesc/X86InstPrinterCommon.cpp
index 8470d26011..8d9fa806a2 100644
--- a/llvm/lib/Target/X86/MCTargetDesc/X86InstPrinterCommon.cpp
+++ b/llvm/lib/Target/X86/MCTargetDesc/X86InstPrinterCommon.cpp
@@ -309,15 +309,24 @@ void X86InstPrinterCommon::printCMPMnemonic(const MCInst *MI, bool IsVCmp,
case X86::VCMPSHZrmik_Int: case X86::VCMPSHZrrik_Int:
OS << "sh\t";
break;
- case X86::VCMPBF16Z128rmi: case X86::VCMPBF16Z128rri:
- case X86::VCMPBF16Z256rmi: case X86::VCMPBF16Z256rri:
- case X86::VCMPBF16Zrmi: case X86::VCMPBF16Zrri:
- case X86::VCMPBF16Z128rmik: case X86::VCMPBF16Z128rrik:
- case X86::VCMPBF16Z256rmik: case X86::VCMPBF16Z256rrik:
- case X86::VCMPBF16Zrmik: case X86::VCMPBF16Zrrik:
- case X86::VCMPBF16Z128rmbi: case X86::VCMPBF16Z128rmbik:
- case X86::VCMPBF16Z256rmbi: case X86::VCMPBF16Z256rmbik:
- case X86::VCMPBF16Zrmbi: case X86::VCMPBF16Zrmbik:
+ case X86::VCMPBF16Z128rmi:
+ case X86::VCMPBF16Z128rri:
+ case X86::VCMPBF16Z256rmi:
+ case X86::VCMPBF16Z256rri:
+ case X86::VCMPBF16Zrmi:
+ case X86::VCMPBF16Zrri:
+ case X86::VCMPBF16Z128rmik:
+ case X86::VCMPBF16Z128rrik:
+ case X86::VCMPBF16Z256rmik:
+ case X86::VCMPBF16Z256rrik:
+ case X86::VCMPBF16Zrmik:
+ case X86::VCMPBF16Zrrik:
+ case X86::VCMPBF16Z128rmbi:
+ case X86::VCMPBF16Z128rmbik:
+ case X86::VCMPBF16Z256rmbi:
+ case X86::VCMPBF16Z256rmbik:
+ case X86::VCMPBF16Zrmbi:
+ case X86::VCMPBF16Zrmbik:
OS << "bf16\t";
break;
}
diff --git a/llvm/lib/Target/X86/MCTargetDesc/X86IntelInstPrinter.cpp b/llvm/lib/Target/X86/MCTargetDesc/X86IntelInstPrinter.cpp
index 0d92609b3a..5a814fce46 100644
--- a/llvm/lib/Target/X86/MCTargetDesc/X86IntelInstPrinter.cpp
+++ b/llvm/lib/Target/X86/MCTargetDesc/X86IntelInstPrinter.cpp
@@ -145,15 +145,24 @@ bool X86IntelInstPrinter::printVecCompareInstr(const MCInst *MI, raw_ostream &OS
case X86::VCMPPHZrmbi: case X86::VCMPPHZrmbik:
case X86::VCMPPHZrrib: case X86::VCMPPHZrribk:
case X86::VCMPSHZrrib_Int: case X86::VCMPSHZrribk_Int:
- case X86::VCMPBF16Z128rmi: case X86::VCMPBF16Z128rri:
- case X86::VCMPBF16Z256rmi: case X86::VCMPBF16Z256rri:
- case X86::VCMPBF16Zrmi: case X86::VCMPBF16Zrri:
- case X86::VCMPBF16Z128rmik: case X86::VCMPBF16Z128rrik:
- case X86::VCMPBF16Z256rmik: case X86::VCMPBF16Z256rrik:
- case X86::VCMPBF16Zrmik: case X86::VCMPBF16Zrrik:
- case X86::VCMPBF16Z128rmbi: case X86::VCMPBF16Z128rmbik:
- case X86::VCMPBF16Z256rmbi: case X86::VCMPBF16Z256rmbik:
- case X86::VCMPBF16Zrmbi: case X86::VCMPBF16Zrmbik:
+ case X86::VCMPBF16Z128rmi:
+ case X86::VCMPBF16Z128rri:
+ case X86::VCMPBF16Z256rmi:
+ case X86::VCMPBF16Z256rri:
+ case X86::VCMPBF16Zrmi:
+ case X86::VCMPBF16Zrri:
+ case X86::VCMPBF16Z128rmik:
+ case X86::VCMPBF16Z128rrik:
+ case X86::VCMPBF16Z256rmik:
+ case X86::VCMPBF16Z256rrik:
+ case X86::VCMPBF16Zrmik:
+ case X86::VCMPBF16Zrrik:
+ case X86::VCMPBF16Z128rmbi:
+ case X86::VCMPBF16Z128rmbik:
+ case X86::VCMPBF16Z256rmbi:
+ case X86::VCMPBF16Z256rmbik:
+ case X86::VCMPBF16Zrmbi:
+ case X86::VCMPBF16Zrmbik:
if (Imm >= 0 && Imm <= 31) {
OS << '\t';
printCMPMnemonic(MI, /*IsVCMP*/true, OS);
diff --git a/llvm/lib/Target/X86/X86InstrFMA3Info.cpp b/llvm/lib/Target/X86/X86InstrFMA3Info.cpp
index d0c508d993..86f6c733bd 100644
--- a/llvm/lib/Target/X86/X86InstrFMA3Info.cpp
+++ b/llvm/lib/Target/X86/X86InstrFMA3Info.cpp
@@ -52,7 +52,7 @@ using namespace llvm;
FMA3GROUP_PACKED_WIDTHS_Z(Name, PH, Attrs) \
FMA3GROUP_PACKED_WIDTHS_ALL(Name, PS, Attrs)
-#define FMA3GROUP_PACKED_BF16(Name, Attrs) \
+#define FMA3GROUP_PACKED_BF16(Name, Attrs) \
FMA3GROUP_PACKED_WIDTHS_Z(Name, BF16, Attrs)
#define FMA3GROUP_SCALAR_WIDTHS_Z(Name, Suf, Attrs) \
@@ -92,10 +92,10 @@ static const X86InstrFMA3Group Groups[] = {
FMA3GROUP_MASKED(Name, Type##Z256##Suf, Attrs) \
FMA3GROUP_MASKED(Name, Type##Z##Suf, Attrs)
-#define FMA3GROUP_PACKED_AVX512_ALL(Name, Suf, Attrs) \
- FMA3GROUP_PACKED_AVX512_WIDTHS(Name, BF16, Suf, Attrs) \
- FMA3GROUP_PACKED_AVX512_WIDTHS(Name, PD, Suf, Attrs) \
- FMA3GROUP_PACKED_AVX512_WIDTHS(Name, PH, Suf, Attrs) \
+#define FMA3GROUP_PACKED_AVX512_ALL(Name, Suf, Attrs) \
+ FMA3GROUP_PACKED_AVX512_WIDTHS(Name, BF16, Suf, Attrs) \
+ FMA3GROUP_PACKED_AVX512_WIDTHS(Name, PD, Suf, Attrs) \
+ FMA3GROUP_PACKED_AVX512_WIDTHS(Name, PH, Suf, Attrs) \
FMA3GROUP_PACKED_AVX512_WIDTHS(Name, PS, Suf, Attrs)
#define FMA3GROUP_PACKED_AVX512_DHS(Name, Suf, Attrs) \
``````````
</details>
https://github.com/llvm/llvm-project/pull/123335
More information about the llvm-commits
mailing list