[PATCH] [AArch64 NEON] Accept both #0.0 and #0 for comparing with floating point zero in asm parser.

Tim Northover t.p.northover at gmail.com
Wed Dec 18 11:03:03 PST 2013


  Hi Kevin,

  A nice change overall, but it may be slightly more complicated than necessary:


================
Comment at: lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:1642
@@ +1641,3 @@
+  }
+  APFloat RealVal(APFloat::IEEEdouble, Parser.getTok().getString());
+  if (Negative) RealVal.changeSign();
----------------
This asserts on "0x0", which is slightly surprising as a user. But given that the only valid value is 0, couldn't this function be rather simplified anyway? And made to give better diagnostics in the process.


http://llvm-reviews.chandlerc.com/D2432



More information about the llvm-commits mailing list