[PATCH] D11914: WebAssembly: print immediates

[Dan Gohman via llvm-commits]

sunfish accepted this revision.
This revision is now accepted and ready to land.

================
Comment at: lib/Target/WebAssembly/WebAssemblyAsmPrinter.cpp:122
@@ +121,3 @@
+          buf, /*hexDigits=*/0, /*upperCase=*/false,
+          APFloat::rmNearestTiesToEven);
+      assert(Written != 0);
----------------
It looks like APFloat's NaN printing doesn't include the significand (the "(n-char-sequence)"). I don't want to bikeshed on the format here, but we'll need to do *something* to preserve all the information in any case. Can you add asserts, or perhaps just highly-visible comments, indicating that we're missing functionality here?


http://reviews.llvm.org/D11914