Skip to content

[NFC][SPARC] Fix typos and style mismatches #96019

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

koachan
Copy link
Contributor

@koachan koachan commented Jun 19, 2024

Fix style errors accidentally introduced in PRs #87259 and #94245.

Created using spr 1.3.5
@llvmbot llvmbot added backend:Sparc mc Machine (object) code labels Jun 19, 2024
@llvmbot
Copy link
Member

llvmbot commented Jun 19, 2024

@llvm/pr-subscribers-backend-sparc

@llvm/pr-subscribers-mc

Author: Koakuma (koachan)

Changes

Fix style errors accidentally introduced in PRs #87259 and #94245.


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

3 Files Affected:

  • (modified) llvm/lib/Target/Sparc/SparcInstrAliases.td (+1-1)
  • (modified) llvm/test/CodeGen/SPARC/inlineasm.ll (+1-1)
  • (modified) llvm/test/MC/Disassembler/Sparc/sparc-v9.txt (+5-5)
diff --git a/llvm/lib/Target/Sparc/SparcInstrAliases.td b/llvm/lib/Target/Sparc/SparcInstrAliases.td
index eedad25737f65..e77b7f5c376bf 100644
--- a/llvm/lib/Target/Sparc/SparcInstrAliases.td
+++ b/llvm/lib/Target/Sparc/SparcInstrAliases.td
@@ -568,7 +568,7 @@ let EmitPriority = 0 in {
   // or imm, reg, rd -> or reg, imm, rd
   def : InstAlias<"or $simm13, $rs1, $rd", (ORri IntRegs:$rd, IntRegs:$rs1, simm13Op:$simm13)>;
 
-  // addc/addx imm, reg, rd -> or reg, imm, rd
+  // addc/addx imm, reg, rd -> addc/addx reg, imm, rd
   def : InstAlias<"addx $simm13, $rs1, $rd", (ADDCri IntRegs:$rd, IntRegs:$rs1, simm13Op:$simm13)>;
 }
 
diff --git a/llvm/test/CodeGen/SPARC/inlineasm.ll b/llvm/test/CodeGen/SPARC/inlineasm.ll
index 9817d7c6971f5..14ea0a2a12602 100644
--- a/llvm/test/CodeGen/SPARC/inlineasm.ll
+++ b/llvm/test/CodeGen/SPARC/inlineasm.ll
@@ -144,7 +144,7 @@ entry:
   ret void
 }
 
-; CHECK-label:test_twinword
+; CHECK-LABEL: test_twinword:
 ; CHECK: rd  %asr5, %i1
 ; CHECK: srlx %i1, 32, %i0
 
diff --git a/llvm/test/MC/Disassembler/Sparc/sparc-v9.txt b/llvm/test/MC/Disassembler/Sparc/sparc-v9.txt
index d561216fec6f2..9e4ff04841d5a 100644
--- a/llvm/test/MC/Disassembler/Sparc/sparc-v9.txt
+++ b/llvm/test/MC/Disassembler/Sparc/sparc-v9.txt
@@ -193,16 +193,16 @@
 0xef 0x6e 0x40 0x1a
 
 # CHECK: done
-0x81,0xf0,0x00,0x00
+0x81 0xf0 0x00 0x00
 
 # CHECK: retry
-0x83,0xf0,0x00,0x00
+0x83 0xf0 0x00 0x00
 
 # CHECK: saved
-0x81,0x88,0x00,0x00
+0x81 0x88 0x00 0x00
 
 # CHECK: restored
-0x83,0x88,0x00,0x00
+0x83 0x88 0x00 0x00
 
 # CHECK: rdpr %fq, %i5
-0xbb,0x53,0xc0,0x00
+0xbb 0x53 0xc0 0x00

@koachan koachan merged commit edd2d7c into main Jun 19, 2024
10 checks passed
@koachan koachan deleted the users/koachan/spr/nfcsparc-fix-typos-and-style-mismatches branch June 19, 2024 14:44
AlexisPerry pushed a commit to llvm-project-tlp/llvm-project that referenced this pull request Jul 9, 2024
Fix style errors accidentally introduced in PRs llvm#87259 and llvm#94245.

Reviewers: rorth, jrtc27, brad0, s-barannikov

Reviewed By: s-barannikov

Pull Request: llvm#96019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend:Sparc mc Machine (object) code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants