[PATCH] D79345: Summary:Fix decimal formatting of 80-bit x87 values; the calculation ofnearest neighbor values failed to account for the explicitmost significant bit in that format.

Peter Klausler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 4 11:50:15 PDT 2020


klausler created this revision.
klausler added reviewers: tskeith, sscalpone.
klausler added a project: Flang.
Herald added a reviewer: jdoerfert.
Herald added a reviewer: DavidTruby.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Replace MultiplyByRounded with MultiplyBy in binary->decimal conversions,
since rounding won't happen and the name was misleading; then remove
dead code, and migrate LoseLeastSignificantDigit() from one source file
to another where it's still needed.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D79345

Files:
  flang/include/flang/Common/uint128.h
  flang/include/flang/Decimal/binary-floating-point.h
  flang/lib/Decimal/big-radix-floating-point.h
  flang/lib/Decimal/binary-to-decimal.cpp
  flang/lib/Decimal/decimal-to-binary.cpp
  flang/runtime/edit-output.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79345.261885.patch
Type: text/x-patch
Size: 10493 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200504/e8452b00/attachment.bin>


More information about the llvm-commits mailing list