[PATCH] D76057: [Reassociate] recognize more than one pairs for later CSE - NFC
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 13 03:41:30 PDT 2020
fhahn added a comment.
I think it would make the review much easier if the description would include the rational behind the changes, i.e how the patch works, why the changes are necessary in terms of implementation and what issue the patch concretely addressed.
Also you mention follow-on changes that build on this one. Sharing them would probably also help with reviewing this change, to see how things fit together in the end. Phabricator lets you link related/dependent patches.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76057/new/
https://reviews.llvm.org/D76057
More information about the llvm-commits
mailing list