[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