diff options
author | Daniel Jasper <djasper@google.com> | 2017-06-29 13:58:24 +0000 |
---|---|---|
committer | Daniel Jasper <djasper@google.com> | 2017-06-29 13:58:24 +0000 |
commit | 559aa75382e941a1c0c2fc5724d5274c20f2adf8 (patch) | |
tree | c8f215d8add480f68484fcc2ef8043b94f967598 /llvm/lib/CodeGen/MachineInstr.cpp | |
parent | 8a44b7be76f35f26d56327952bd133a48c07f301 (diff) | |
download | bcm5719-llvm-559aa75382e941a1c0c2fc5724d5274c20f2adf8.tar.gz bcm5719-llvm-559aa75382e941a1c0c2fc5724d5274c20f2adf8.zip |
Revert "r306529 - [X86] Correct dwarf unwind information in function epilogue"
I am 99% sure that this breaks the PPC ASAN build bot:
http://lab.llvm.org:8011/builders/sanitizer-ppc64be-linux/builds/3112/steps/64-bit%20check-asan/logs/stdio
If it doesn't go back to green, we can recommit (and fix the original
commit message at the same time :) ).
llvm-svn: 306676
Diffstat (limited to 'llvm/lib/CodeGen/MachineInstr.cpp')
-rw-r--r-- | llvm/lib/CodeGen/MachineInstr.cpp | 38 |
1 files changed, 4 insertions, 34 deletions
diff --git a/llvm/lib/CodeGen/MachineInstr.cpp b/llvm/lib/CodeGen/MachineInstr.cpp index 01fb2a19779..81c6dace92e 100644 --- a/llvm/lib/CodeGen/MachineInstr.cpp +++ b/llvm/lib/CodeGen/MachineInstr.cpp @@ -305,33 +305,8 @@ bool MachineOperand::isIdenticalTo(const MachineOperand &Other) const { } case MachineOperand::MO_MCSymbol: return getMCSymbol() == Other.getMCSymbol(); - case MachineOperand::MO_CFIIndex: { - const MachineFunction *MF = getParent()->getParent()->getParent(); - const MachineFunction *OtherMF = - Other.getParent()->getParent()->getParent(); - MCCFIInstruction Inst = MF->getFrameInstructions()[getCFIIndex()]; - MCCFIInstruction OtherInst = - OtherMF->getFrameInstructions()[Other.getCFIIndex()]; - MCCFIInstruction::OpType op = Inst.getOperation(); - if (op != OtherInst.getOperation()) return false; - if (op == MCCFIInstruction::OpDefCfa || op == MCCFIInstruction::OpOffset || - op == MCCFIInstruction::OpRestore || - op == MCCFIInstruction::OpUndefined || - op == MCCFIInstruction::OpSameValue || - op == MCCFIInstruction::OpDefCfaRegister || - op == MCCFIInstruction::OpRelOffset || - op == MCCFIInstruction::OpRegister) - if (Inst.getRegister() != OtherInst.getRegister()) return false; - if (op == MCCFIInstruction::OpRegister) - if (Inst.getRegister2() != OtherInst.getRegister2()) return false; - if (op == MCCFIInstruction::OpDefCfa || op == MCCFIInstruction::OpOffset || - op == MCCFIInstruction::OpRelOffset || - op == MCCFIInstruction::OpDefCfaOffset || - op == MCCFIInstruction::OpAdjustCfaOffset || - op == MCCFIInstruction::OpGnuArgsSize) - if (Inst.getOffset() != OtherInst.getOffset()) return false; - return true; - } + case MachineOperand::MO_CFIIndex: + return getCFIIndex() == Other.getCFIIndex(); case MachineOperand::MO_Metadata: return getMetadata() == Other.getMetadata(); case MachineOperand::MO_IntrinsicID: @@ -380,13 +355,8 @@ hash_code llvm::hash_value(const MachineOperand &MO) { return hash_combine(MO.getType(), MO.getTargetFlags(), MO.getMetadata()); case MachineOperand::MO_MCSymbol: return hash_combine(MO.getType(), MO.getTargetFlags(), MO.getMCSymbol()); - case MachineOperand::MO_CFIIndex: { - const MachineFunction *MF = MO.getParent()->getParent()->getParent(); - MCCFIInstruction Inst = MF->getFrameInstructions()[MO.getCFIIndex()]; - return hash_combine(MO.getType(), MO.getTargetFlags(), Inst.getOperation(), - Inst.getRegister(), Inst.getRegister2(), - Inst.getOffset()); - } + case MachineOperand::MO_CFIIndex: + return hash_combine(MO.getType(), MO.getTargetFlags(), MO.getCFIIndex()); case MachineOperand::MO_IntrinsicID: return hash_combine(MO.getType(), MO.getTargetFlags(), MO.getIntrinsicID()); case MachineOperand::MO_Predicate: |