[PATCH] D129612: [Reassociate] Cleanup minor missed optimizations

Warren Ristow via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 07:40:38 PDT 2022


wristow added a comment.

In D129612#3652041 <https://reviews.llvm.org/D129612#3652041>, @spatel wrote:

> I'd prefer to have that test, so we have coverage for the integer path added with this patch. It's still an improvement even if it's not as big as the FP side.
> Other than that, LGTM.

I was just thinking the same thing and adding that test (with a TODO)!


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

https://reviews.llvm.org/D129612



More information about the llvm-commits mailing list