[CodeGen] Make MachineInstr::isIdenticalTo() symmetric.
authorBjorn Pettersson <bjorn.a.pettersson@ericsson.com>
Mon, 19 Dec 2016 11:20:57 +0000 (11:20 +0000)
committerBjorn Pettersson <bjorn.a.pettersson@ericsson.com>
Mon, 19 Dec 2016 11:20:57 +0000 (11:20 +0000)
commitb29a15ecad3f1374827820db66bd872dae89ee0e
treee2b995f546db95813efd0345e690152ea632bc9d
parent7aacf468c013cd8a4e5cb25ba10d231f0c22ccdd
[CodeGen] Make MachineInstr::isIdenticalTo() symmetric.

Summary:
MachineInstr::isIdenticalTo() is for some reason not
symmetric when comparing bundles, which gives us the
property:

  I1->isIdenticalTo(*I2) != I2->isIdenticalTo(*I1)

when comparing bundles where one bundle is longer than
the other.

This patch makes sure that bundles of different length
always are considered as not being identical. Thus, the
result of the comparison will be the same regardless of
which side that happens to be to the left.

Reviewers: dexonsmith, jonpa, andrew.w.kaylor

Subscribers: llvm-commits, mehdi_amini

Differential Revision: https://reviews.llvm.org/D27508

llvm-svn: 290095
llvm/lib/CodeGen/MachineInstr.cpp