[PATCH] D15375: Fix PR #25788: parsing of floating-point constants on non-C locales

Antoine Pitrou via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 28 00:09:22 PST 2016


pitrou added a comment.

> I think we'd want APFloat::convertFromDecimalString with rmNearestTiesToEven seeing as how we don't expect to see -nan and the like.


I've looked at this and there are a couple issues:

- convertFromDecimalString() is private
- convertFromDecimalString() doesn't handle leading sign

Also, due to the lexing that occurs before calling APFloat() here, I don't think it can actually be called with `-nan` and friends. I'll upload an updated patch as it turns out the one here doesn't apply anymore.


http://reviews.llvm.org/D15375





More information about the llvm-commits mailing list