[llvm] [AArch64] Correct defs and uses on {PAC, AUT}I{A, B}171615 (PR #123354)

via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 17 07:22:05 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-backend-aarch64

Author: Kristof Beyls (kbeyls)

<details>
<summary>Changes</summary>

I'm not adding tests for this, as I don't think we usually have tests to verify correct description of defs and uses in instructions?

This fix will be tested when #<!-- -->122304 lands, as one of the regression tests in that PR fails without this fix.

---
Full diff: https://github.com/llvm/llvm-project/pull/123354.diff


1 Files Affected:

- (modified) llvm/lib/Target/AArch64/AArch64InstrInfo.td (+2) 


``````````diff
diff --git a/llvm/lib/Target/AArch64/AArch64InstrInfo.td b/llvm/lib/Target/AArch64/AArch64InstrInfo.td
index 8215f3a4fdae1e..40a6b8e4c8e640 100644
--- a/llvm/lib/Target/AArch64/AArch64InstrInfo.td
+++ b/llvm/lib/Target/AArch64/AArch64InstrInfo.td
@@ -2030,6 +2030,8 @@ let Predicates = [HasPAuthLR] in {
     //                              opcode2, opcode,   asm
     def AUTIASPPCr : SignAuthOneReg<0b00001, 0b100100, "autiasppcr">;
     def AUTIBSPPCr : SignAuthOneReg<0b00001, 0b100101, "autibsppcr">;
+  }
+  let Defs = [X17], Uses = [X15, X16, X17] in {
     //                                  opcode2, opcode,   asm
     def PACIA171615 : SignAuthFixedRegs<0b00001, 0b100010, "pacia171615">;
     def PACIB171615 : SignAuthFixedRegs<0b00001, 0b100011, "pacib171615">;

``````````

</details>


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


More information about the llvm-commits mailing list