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

[Hiroshi Yamauchi via Phabricator via llvm-commits]

yamauchi added a comment.



> Thanks for finding that thread...somehow I find it difficult to search llvm-dev history...

No problem. It was in my email archives.

> I agree that it's not always clear (and note that I posted an alternative version of https://reviews.llvm.org/D45842 in PR37098 that would do more reassociation in instcombine...it would be easier!).

Maybe I do not see the full context of PR37098 but I wonder if there was an actual objection to the instcombine version of https://reviews.llvm.org/D45842.


Repository:
  rL LLVM

https://reviews.llvm.org/D46336