[clang] [compiler-rt] [llvm] Reland "[X86][AVX10.2] Support AVX10.2 option and VMPSADBW/VADDP[D,H,S] new instructions (#101452)" (PR #101616)

Phoebe Wang via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 1 23:57:25 PDT 2024


================
@@ -223,6 +227,10 @@ InstructionContext RecognizableInstr::insnContext() const {
           insnContext = EVEX_KB_U(IC_EVEX_XD);
         else if (OpPrefix == X86Local::PS)
           insnContext = EVEX_KB_U(IC_EVEX);
+        else {
+          errs() << "Instruction does not use a prefix: " << Name << "\n";
+          llvm_unreachable("Invalid prefix");
----------------
phoebewang wrote:

Do you mean remove the `errs() << ...`? How about the others? Both VEX and EVEX have the 4 prefixes.

https://github.com/llvm/llvm-project/pull/101616


More information about the cfe-commits mailing list