[PATCH] D11537: WebAssembly: minor MCAsmInfo fixes

JF Bastien jfb at chromium.org
Mon Jul 27 15:41:24 PDT 2015


jfb marked an inline comment as done.
jfb added a comment.

Follow-up in http://reviews.llvm.org/D11544.


================
Comment at: llvm/trunk/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCAsmInfo.cpp:38
@@ +37,3 @@
+  InlineAsmStart = ";APP\n";
+  InlineAsmEnd = ";NO_APP\n";
+
----------------
sunfish wrote:
> It looks like asm strings are by default printed with emitRawComment which includes the comment character and newline. Do we need to override the defaults for these strings?
Ah you're right, fixed in follow-up.


Repository:
  rL LLVM

http://reviews.llvm.org/D11537







More information about the llvm-commits mailing list