[PATCH] D145645: [RISCV] Print Zfa fli instruction FP values in a more adaptive way.

Jun Sha via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 16 02:10:31 PDT 2023


joshua-arch1 added a comment.

In D145645#4198610 <https://reviews.llvm.org/D145645#4198610>, @craig.topper wrote:

> In D145645#4198567 <https://reviews.llvm.org/D145645#4198567>, @majin2020 wrote:
>
>> hi,
>> I don't quite understand why this is necessary. Isn't it more reasonable to have a unified format? The cost is very small.
>
>
>
> In D145645#4198567 <https://reviews.llvm.org/D145645#4198567>, @majin2020 wrote:
>
>> hi,
>> I don't quite understand why this is necessary. Isn't it more reasonable to have a unified format? The cost is very small.
>
> We need 11 digits of decimal point for the smallest number consistency made the other numbers overly long.

I think we can use less digits of decimal point for values other than 1*2^-15 and 1*2^-16, but with scientific notation consistently.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D145645/new/

https://reviews.llvm.org/D145645



More information about the llvm-commits mailing list