[PATCH] D27508: [CodeGen] Make MachineInstr::isIdenticalTo() symmetric.
Andy Kaylor via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 16 10:59:18 PST 2016
andrew.w.kaylor accepted this revision.
andrew.w.kaylor added a comment.
This revision is now accepted and ready to land.
I have one nitpick about a typo in a comment, but otherwise this looks good to me.
================
Comment at: lib/CodeGen/MachineInstr.cpp:1003
if (isBundle()) {
- // Both instructions are bundles, compare MIs inside the bundle.
+ // We have passed the test above than both instructions have the same
+ // opcode, so we know that both instructions are bundles here. Let's compare
----------------
s/than/that
https://reviews.llvm.org/D27508
More information about the llvm-commits
mailing list