[PATCH] D82730: [SimplifyCFG] Merge identical basic blocks (WIP)

Andrew Litteken via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 30 10:18:19 PDT 2020


AndrewLitteken added a comment.

We're using a similar method for how to do the Instruction equivalence checking by using `isSameOperationAs` for most of the checking, but is there a reason you are not using `isIdenticalTo` when checking that instructions are equivalent?  It looks like it is doing much of the same thing, including checking if operands and phi nodes are the same.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82730/new/

https://reviews.llvm.org/D82730





More information about the llvm-commits mailing list