[llvm] [RISCV] Loosen the requirement of shadow stack codegen to Zimop (PR #152251)
Ming-Yi Lai via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 5 23:54:45 PDT 2025
================
@@ -14,40 +14,40 @@
# CHECK-ASM-AND-OBJ: c.sspopchk t0
# CHECK-ASM: encoding: [0x81,0x62]
-# CHECK-NO-EXT: error: instruction requires the following: 'Zicfiss' (Shadow stack)
+# CHECK-NO-EXT: error: instruction requires the following: 'Zimop' (May-Be-Operations){{$}}
----------------
mylai-mtk wrote:
Excuse me, but I would say that the original Zicfiss message should be kept intact. It should not be changed to Zimop nor Zcmop.
It looks like this test takes in Zicfiss insns and outputs its corresponding compressed form, and shouldn't the action of "taking in Zicfiss insns" require Zicfiss instead of Zimop? Zimop/Zcmop do not define these shadow stack insns and it should be an error to use these insns without Zicfiss.
https://github.com/llvm/llvm-project/pull/152251
More information about the llvm-commits
mailing list