[PATCH] D129715: [LoongArch] Heuristically load FP immediates by movgr2fr from materialized integer

Wed Jul 27 18:57:56 PDT 2022

gonglingqin added a comment.

In D129715#3682174 <https://reviews.llvm.org/D129715#3682174>, @xry111 wrote:

>> I think @xry111's suggestion to make this optimization tunable is reasonable, but it may need more work then. Perhaps this patch could be put on the back burner, we can always come to finish this later after the more essential things. Lots of downstream projects are blocked by availability of LLVM so it may be very worthwhile to shift priorities for now.
>
> Yes, if there is too much extra cost we can just delay this one.  My suggestion to make a tunable is based on "the logic is already written and let's not waste it".

Thank you for your suggestions. After discussion, we will continue to improve this patch after the implementation of important functions.

> Apologize again for raising some premature thoughts too early :(.

You don't have to feel guilty. This is an interesting optimization.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129715