[PATCH] D87401: [ms] [llvm-ml] Add MASM hex float support

[Nico Weber via Phabricator via llvm-commits]

thakis accepted this revision.
thakis added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/MC/MCParser/MasmParser.cpp:3455
+    // MASM hexadecimal floating-point literal; no APFloat conversion needed.
+    // Also, to match ML64.exe, ignore the initial sign.
+    unsigned Size = Value.getSizeInBits(Semantics);
----------------
Should we at least warn about an explicit sign with a hexfloat?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87401/new/

https://reviews.llvm.org/D87401