[PATCH] D11537: WebAssembly: minor MCAsmInfo fixes

JF Bastien jfb at chromium.org
Mon Jul 27 13:47:19 PDT 2015


This revision was automatically updated to reflect the committed changes.
Closed by commit rL243326: WebAssembly: minor MCAsmInfo fixes (authored by jfb).

Changed prior to commit:
  http://reviews.llvm.org/D11537?vs=30730&id=30732#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D11537

Files:
  llvm/trunk/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCAsmInfo.cpp

Index: llvm/trunk/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCAsmInfo.cpp
===================================================================
--- llvm/trunk/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCAsmInfo.cpp
+++ llvm/trunk/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCAsmInfo.cpp
@@ -23,13 +23,20 @@
 WebAssemblyMCAsmInfo::~WebAssemblyMCAsmInfo() {}
 
 WebAssemblyMCAsmInfo::WebAssemblyMCAsmInfo(const Triple &T) {
-  PointerSize = CalleeSaveStackSlotSize = T.isArch64Bit();
+  PointerSize = CalleeSaveStackSlotSize = T.isArch64Bit() ? 8 : 4;
 
   // TODO: What should MaxInstLength be?
 
+  // WebAssembly's text format uses s-expressions to represent its AST,
+  // LISP-style comments are therefore suitable.
+  CommentString = ";";
+
   PrivateGlobalPrefix = "";
   PrivateLabelPrefix = "";
 
+  InlineAsmStart = ";APP\n";
+  InlineAsmEnd = ";NO_APP\n";
+
   UseDataRegionDirectives = true;
 
   Data8bitsDirective = "\t.int8\t";


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11537.30732.patch
Type: text/x-patch
Size: 953 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150727/3debf213/attachment.bin>


More information about the llvm-commits mailing list