[PATCH] D57321: Fix LexFloatLiteral Lexing

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 15 14:38:46 PDT 2019


efriedma added inline comments.


================
Comment at: llvm/test/MC/AsmParser/floating-literals.s:60
+# CHECK-ERROR: Invalid sign in float literal
+.double 2.+1
 
----------------
efriedma wrote:
> We should probably have testcases for 1E1, 1e1e1, and 1e-1, since those don't work correctly without this patch.
In this context, 1E1 is different from 1e1...

Probably best to check all of these with lowercase and uppercase "E".


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57321





More information about the llvm-commits mailing list