<div dir="ltr"><div>I'm not sure this is actually a good idea. The asm file in question seems to me probably to have just been a typo. (Says %0 in one place, instead of %g0 or 0.)</div><div><br></div><div>Unless there's more widespread usage of this, I think it might be better to treat that "%0" usage as a bug, and just fix it, rather than adding support for this confusing register-naming variant in llvm.</div><div><br></div><div>Solaris 'as' also doesn't accept that syntax.</div><div>$ echo "mov %0, %g1" > /tmp/test.s</div><div>$ as -o /tmp/test.o /tmp/test.s</div><div>as: "/tmp/test.s", line 1: error: statement syntax<br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 21, 2015 at 10:34 PM, Douglas Katzman <span dir="ltr"><<a href="mailto:dougk@google.com" target="_blank">dougk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">dougk created this revision.<br>
dougk added a reviewer: jyknight.<br>
dougk added a subscriber: llvm-commits.<br>
<br>
This syntax appears in actual code in the wild, and is valid according to gnu as.<br>
<br>
<a href="http://reviews.llvm.org/D12261" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12261</a><br>
<br>
Files:<br>
  lib/Target/Sparc/AsmParser/SparcAsmParser.cpp<br>
  test/MC/Sparc/sparc-alu-instructions.s<br>
<br>
Index: test/MC/Sparc/sparc-alu-instructions.s<br>
===================================================================<br>
--- test/MC/Sparc/sparc-alu-instructions.s<br>
+++ test/MC/Sparc/sparc-alu-instructions.s<br>
@@ -126,3 +126,7 @@<br>
<br>
         ! CHECK:  tsubcctv %g2, %g1, %g3          ! encoding: [0x87,0x18,0x80,0x01]<br>
         tsubcctv %g2, %g1, %g3<br>
+<br>
+        ! "%0" is a synonym for "%g0" and so on<br>
+        ! CHECK:   mov  %g0, %i7                 ! encoding: [0xbe,0x10,0x00,0x00]<br>
+        mov %0, %31<br>
Index: lib/Target/Sparc/AsmParser/SparcAsmParser.cpp<br>
===================================================================<br>
--- lib/Target/Sparc/AsmParser/SparcAsmParser.cpp<br>
+++ lib/Target/Sparc/AsmParser/SparcAsmParser.cpp<br>
@@ -1023,6 +1023,19 @@<br>
       return true;<br>
     }<br>
   }<br>
+  // For compatibility with other assemblers, %0 through %31 refer to integer<br>
+  // registers without subdividing them into globals,locals,outs,ins.<br>
+  // Strictly speaking, the valid spellings exclude alternative ways of<br>
+  // writing the decimal integers, but the lexer doesn't know that.<br>
+  if (Tok.is(AsmToken::Integer)) {<br>
+    intVal = Tok.getIntVal();<br>
+    // The size() check rejects "%001", "%0b1", "%0x1" as putative spellings.<br>
+    if (intVal >= 0 && intVal < 32 && Tok.getString().size()<= 2) {<br>
+        RegNo = IntRegs[intVal];<br>
+        RegKind = SparcOperand::rk_IntReg;<br>
+        return true;<br>
+    }<br>
+  }<br>
   return false;<br>
 }<br>
<br>
<br>
<br>
</blockquote></div><br></div>