[PATCH] D11544: WebAssembly: more MCAsmInfo nits.

JF Bastien jfb at chromium.org
Mon Jul 27 15:37:58 PDT 2015


jfb created this revision.
jfb added subscribers: sunfish, llvm-commits.
Herald added a subscriber: jfb.

As suggested by sunfish.

http://reviews.llvm.org/D11544

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

Index: lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCAsmInfo.cpp
===================================================================
--- lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCAsmInfo.cpp
+++ lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCAsmInfo.cpp
@@ -23,10 +23,13 @@
 WebAssemblyMCAsmInfo::~WebAssemblyMCAsmInfo() {}
 
 WebAssemblyMCAsmInfo::WebAssemblyMCAsmInfo(const Triple &T) {
-  PointerSize = CalleeSaveStackSlotSize = T.isArch64Bit();
+  PointerSize = CalleeSaveStackSlotSize = T.isArch64Bit() ? 8 : 4;
 
   // TODO: What should MaxInstLength be?
 
+  // The s-expression format of WebAssembly uses LISP-style comments.
+  CommentString = ";";
+
   PrivateGlobalPrefix = "";
   PrivateLabelPrefix = "";
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11544.30747.patch
Type: text/x-patch
Size: 731 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150727/0ad61654/attachment.bin>


More information about the llvm-commits mailing list