[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