[PATCH] D65954: [Reassociate] try harder to convert negative FP constants to positive

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 8 13:25:59 PDT 2019


spatel added a comment.

In D65954#1621623 <https://reviews.llvm.org/D65954#1621623>, @mcberg2017 wrote:

> We have some stir(some regressions - a few instructions here and there), however the change is overall beneficial for us.
>
> LGTM.


Thanks! If you can share any of those tests that regressed, I can dig further into this pass to see if we can improve them.


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

https://reviews.llvm.org/D65954





More information about the llvm-commits mailing list