[PATCH] D57321: Fix LexFloatLiteral Lexing

Brandon Jones via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 21 05:50:43 PST 2019


BrandonTJones marked an inline comment as done.
BrandonTJones added inline comments.


================
Comment at: llvm/lib/MC/MCParser/AsmLexer.cpp:73
+  if (*CurPtr == '-' || *CurPtr == '+')
+    return ReturnError(CurPtr, "Invalid sign in float literal");
+  // Check for exponent; we enforce rigidity on a correct
----------------
efriedma wrote:
> Is this early return necessary, or just to try to improve the error messages?
Improve the error message, it feels ambiguous to allow the error message to be from the parser not expecting 2 floats in a row.


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