[llvm] [SPARC][IAS] Add `illtrap` alias for `unimp` (PR #105928)
via llvm-commits
llvm-commits at lists.llvm.org
Sat Aug 24 00:26:15 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-sparc
Author: Koakuma (koachan)
<details>
<summary>Changes</summary>
This follows GNU and Solaris behavior of allowing both mnemonics all the time.
Fixes https://github.com/llvm/llvm-project/issues/105639.
---
Full diff: https://github.com/llvm/llvm-project/pull/105928.diff
2 Files Affected:
- (modified) llvm/lib/Target/Sparc/SparcInstrAliases.td (+4)
- (modified) llvm/test/MC/Sparc/sparc-misc-instructions.s (+6)
``````````diff
diff --git a/llvm/lib/Target/Sparc/SparcInstrAliases.td b/llvm/lib/Target/Sparc/SparcInstrAliases.td
index e77b7f5c376bf4..722e0463b7bcaa 100644
--- a/llvm/lib/Target/Sparc/SparcInstrAliases.td
+++ b/llvm/lib/Target/Sparc/SparcInstrAliases.td
@@ -601,6 +601,10 @@ def : InstAlias<"flush", (FLUSH), 0>;
// unimp -> unimp 0
def : InstAlias<"unimp", (UNIMP 0), 0>;
+// Not in spec, but we follow GNU & Solaris behavior of having `illtrap`
+// interchangeable with `unimp` all the time.
+def : MnemonicAlias<"illtrap", "unimp">;
+
def : MnemonicAlias<"iflush", "flush">;
def : MnemonicAlias<"stub", "stb">;
diff --git a/llvm/test/MC/Sparc/sparc-misc-instructions.s b/llvm/test/MC/Sparc/sparc-misc-instructions.s
index 0547575eb3db51..8119088cd0dbb1 100644
--- a/llvm/test/MC/Sparc/sparc-misc-instructions.s
+++ b/llvm/test/MC/Sparc/sparc-misc-instructions.s
@@ -6,3 +6,9 @@
! CHECK: unimp 0 ! encoding: [0x00,0x00,0x00,0x00]
unimp 0
+
+ ! CHECK: unimp 0 ! encoding: [0x00,0x00,0x00,0x00]
+ illtrap
+
+ ! CHECK: unimp 0 ! encoding: [0x00,0x00,0x00,0x00]
+ illtrap 0
``````````
</details>
https://github.com/llvm/llvm-project/pull/105928
More information about the llvm-commits
mailing list