summaryrefslogtreecommitdiffstats
path: root/llvm/lib/CodeGen/BranchFolding.cpp
diff options
context:
space:
mode:
authorBill Wendling <isanbard@gmail.com>2007-10-25 18:23:45 +0000
committerBill Wendling <isanbard@gmail.com>2007-10-25 18:23:45 +0000
commit5f7ed00d4490cc3811e20c5dc3051e81b2d34601 (patch)
treec5a808cc18b9f3d5821fd0700596baa876f4294a /llvm/lib/CodeGen/BranchFolding.cpp
parent0cf94e9c91565c1ab9f4232958309556f9819ee8 (diff)
downloadbcm5719-llvm-5f7ed00d4490cc3811e20c5dc3051e81b2d34601.tar.gz
bcm5719-llvm-5f7ed00d4490cc3811e20c5dc3051e81b2d34601.zip
Added comment explaining why we are doing this check.
llvm-svn: 43353
Diffstat (limited to 'llvm/lib/CodeGen/BranchFolding.cpp')
-rw-r--r--llvm/lib/CodeGen/BranchFolding.cpp5
1 files changed, 5 insertions, 0 deletions
diff --git a/llvm/lib/CodeGen/BranchFolding.cpp b/llvm/lib/CodeGen/BranchFolding.cpp
index d523bf585ae..021001d15fc 100644
--- a/llvm/lib/CodeGen/BranchFolding.cpp
+++ b/llvm/lib/CodeGen/BranchFolding.cpp
@@ -272,6 +272,11 @@ static unsigned ComputeCommonTailLength(MachineBasicBlock *MBB1,
while (I1 != MBB1->begin() && I2 != MBB2->begin()) {
--I1; --I2;
if (!I1->isIdenticalTo(I2) ||
+ // XXX: This check is dubious. It's used to get around a problem where
+ // people incorrectly expect inline asm directives to remain in the same
+ // relative order. This is untenable because normal compiler
+ // optimizations (like this one) may reorder and/or merge these
+ // directives.
I1->getOpcode() == TargetInstrInfo::INLINEASM) {
++I1; ++I2;
break;
OpenPOWER on IntegriCloud