[PATCH] D46336: [InstCombine] Apply binary operator simplifications to associative/commutative cases.

Hiroshi Yamauchi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 8 10:13:22 PDT 2018


yamauchi added a comment.

In https://reviews.llvm.org/D46336#1090655, @lebedev.ri wrote:

> I'd guess something in instcombine does not use commutative matchers.
>  (I did not analyse this at all yet, just 'saving' it as one comment)
>  https://rise4fun.com/Alive/PsC


It's not immediately obvious to me which matcher this may relate to, if it's matcher issue.

Could this also be what this patch aims to explore, ie. folding a commutative alternative position of the nested or's (%4/%2) above?


Repository:
  rL LLVM

https://reviews.llvm.org/D46336





More information about the llvm-commits mailing list