[WebAssembly] Use the new offset syntax for memory operands in inline asm.

Dan Gohman · Dan Gohman · commit e2831b4e27f0 · 2015-12-16T18:14:49.000Z
llvm-svn: 255788
diff --git a/llvm/lib/Target/WebAssembly/WebAssemblyAsmPrinter.cpp b/llvm/lib/Target/WebAssembly/WebAssemblyAsmPrinter.cpp
@@ -260,7 +260,9 @@ bool WebAssemblyAsmPrinter::PrintAsmMemoryOperand(const MachineInstr *MI,
     report_fatal_error("There are no defined alternate asm variants");
 
   if (!ExtraCode) {
-    OS << regToString(MI->getOperand(OpNo));
+    // TODO: For now, we just hard-code 0 as the constant offset; teach
+    // SelectInlineAsmMemoryOperand how to do address mode matching.
+    OS << "0(" + regToString(MI->getOperand(OpNo)) + ')';
     return false;
   }
 
diff --git a/llvm/test/CodeGen/WebAssembly/inline-asm.ll b/llvm/test/CodeGen/WebAssembly/inline-asm.ll
@@ -21,7 +21,7 @@ entry:
 ; CHECK-LABEL: bar:
 ; CHECK-NEXT: .param i32, i32{{$}}
 ; CHECK-NEXT: #APP{{$}}
-; CHECK-NEXT: # $1 = bbb($0){{$}}
+; CHECK-NEXT: # 0($1) = bbb(0($0)){{$}}
 ; CHECK-NEXT: #NO_APP{{$}}
 ; CHECK-NEXT: return{{$}}
 define void @bar(i32* %r, i32* %s) {

Original file line number	Diff line number	Diff line change
`@@ -260,7 +260,9 @@ bool WebAssemblyAsmPrinter::PrintAsmMemoryOperand(const MachineInstr *MI,`
`260`	`260`	`report_fatal_error("There are no defined alternate asm variants");`
`261`	`261`
`262`	`262`	`if (!ExtraCode) {`
`263`		`- OS << regToString(MI->getOperand(OpNo));`
	`263`	`+ // TODO: For now, we just hard-code 0 as the constant offset; teach`
	`264`	`+ // SelectInlineAsmMemoryOperand how to do address mode matching.`
	`265`	`+ OS << "0(" + regToString(MI->getOperand(OpNo)) + ')';`
`264`	`266`	`return false;`
`265`	`267`	`}`
`266`	`268`