[PATCH] D132783: [InstCombine] add support for multi-use Y of (X op Y) op Z --> (Y op Z) op X
Chenbing.Zheng via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Aug 28 19:57:31 PDT 2022
Chenbing.Zheng added a comment.
In D132783#3754025 <https://reviews.llvm.org/D132783#3754025>, @bcl5980 wrote:
> I agree this change help to get more opportune to combine. But I still feel a little strange for the non-deterministic transform here.
Yer, this change will not directly optimizeļ¼and the information provided here is limited. When X, Y are both multi-use, We don't know how to transform so that the code is optimized.
But at least this provides more opportunities here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132783/new/
https://reviews.llvm.org/D132783
More information about the llvm-commits
mailing list