[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