[PATCH] D128269: Teach fpcmp about the x. FP format

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 21 05:59:17 PDT 2022


rovka created this revision.
rovka added reviewers: awarzynski, kiranchandramohan, Meinersbur, sscalpone, naromero77.
rovka added a project: Flang.
Herald added a subscriber: jdoerfert.
Herald added a project: All.
rovka requested review of this revision.

At the moment `fpcmp` rejects floating point numbers with no decimal
digits after the decimal point, such as "3.". However, `flang-new` often
prints floating point numbers in this format and it seems to be
a perfectly respectable format (`strtod` is happy enough to gobble it up).

This patch updates `fpcmp` to allow the absence of post-decimal digits. It
also adds a new command line flag, '-d', that preserves the previous
behaviour. AFAICT nothing in the test-suite needs it, but I haven't
tried any of the External suites.


Repository:
  rT test-suite

https://reviews.llvm.org/D128269

Files:
  tools/fpcmp.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D128269.438670.patch
Type: text/x-patch
Size: 3537 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220621/61f3f694/attachment.bin>


More information about the llvm-commits mailing list