[all-commits] [llvm/llvm-project] e99d18: [flang] Readability improvement in binary->decimal...

Peter Klausler via All-commits all-commits at lists.llvm.org
Thu Oct 1 15:50:20 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: e99d184d54937b56d5f4f1ba06fb984019beaee1
      https://github.com/llvm/llvm-project/commit/e99d184d54937b56d5f4f1ba06fb984019beaee1
  Author: peter klausler <pklausler at nvidia.com>
  Date:   2020-10-01 (Thu, 01 Oct 2020)

  Changed paths:
    R flang/include/flang/Common/unsigned-const-division.h
    M flang/lib/Decimal/big-radix-floating-point.h
    M flang/lib/Decimal/binary-to-decimal.cpp
    M flang/runtime/edit-output.cpp

  Log Message:
  -----------
  [flang] Readability improvement in binary->decimal conversion

Tweak binary->decimal conversions to avoid an integer multiplication
in a hot loop to improve readability and get a minor (~5%) speed-up.
Use native integer division by constants for more readability, too,
since current build compilers seem to optimize it correctly now.
Delete the now needless temporary work-around facility in
Common/unsigned-const-division.h.

Differential revision: https://reviews.llvm.org/D88604




More information about the All-commits mailing list