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

Xi Ruoyao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 27 06:15:23 PDT 2022


xry111 added a comment.

> 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".

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


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129715



More information about the llvm-commits mailing list