diff options
author | Bob Haarman <llvm@inglorion.net> | 2019-05-21 11:53:41 +0000 |
---|---|---|
committer | Bob Haarman <llvm@inglorion.net> | 2019-05-21 11:53:41 +0000 |
commit | 032f87bbb3c04ea4c8287fe339fe047fbff7aae1 (patch) | |
tree | 4119a2cd4df7379b7a14e83f5b9c96e83e8f491f /llvm/lib/Transforms/Utils/Local.cpp | |
parent | 116e8d4876f9745119671c99c12924129526237b (diff) | |
download | llvm-032f87bbb3c04ea4c8287fe339fe047fbff7aae1.zip llvm-032f87bbb3c04ea4c8287fe339fe047fbff7aae1.tar.gz llvm-032f87bbb3c04ea4c8287fe339fe047fbff7aae1.tar.bz2 |
Revert r360902 "Resubmit: [Salvage] Change salvage debug info ..."
This reverts commit rr360902. It caused an assertion failure in
lib/IR/DebugInfoMetadata.cpp: Assertion `(OffsetInBits + SizeInBits <=
FragmentSizeInBits) && "new fragment outside of original fragment"'
failed.
PR41931.
llvm-svn: 361246
Diffstat (limited to 'llvm/lib/Transforms/Utils/Local.cpp')
-rw-r--r-- | llvm/lib/Transforms/Utils/Local.cpp | 23 |
1 files changed, 2 insertions, 21 deletions
diff --git a/llvm/lib/Transforms/Utils/Local.cpp b/llvm/lib/Transforms/Utils/Local.cpp index d6dbb91..91d33cb 100644 --- a/llvm/lib/Transforms/Utils/Local.cpp +++ b/llvm/lib/Transforms/Utils/Local.cpp @@ -1690,27 +1690,8 @@ DIExpression *llvm::salvageDebugInfoImpl(Instruction &I, // No-op casts and zexts are irrelevant for debug info. if (CI->isNoopCast(DL) || isa<ZExtInst>(&I)) return SrcDIExpr; - - Type *Type = CI->getType(); - // Casts other than Trunc or SExt to scalar types cannot be salvaged. - if (Type->isVectorTy() || (!isa<TruncInst>(&I) && !isa<SExtInst>(&I))) - return nullptr; - - Value *FromValue = CI->getOperand(0); - unsigned FromTypeBitSize = FromValue->getType()->getScalarSizeInBits(); - - unsigned ToTypeBitSize = Type->getScalarSizeInBits(); - - // The result of the cast will be sign extended iff the instruction is a - // SExt; signedness is otherwise irrelevant on the expression stack. - unsigned Encoding = - isa<SExtInst>(&I) ? dwarf::DW_ATE_signed : dwarf::DW_ATE_unsigned; - - return applyOps({dwarf::DW_OP_LLVM_convert, FromTypeBitSize, Encoding, - dwarf::DW_OP_LLVM_convert, ToTypeBitSize, Encoding}); - } - - if (auto *GEP = dyn_cast<GetElementPtrInst>(&I)) { + return nullptr; + } else if (auto *GEP = dyn_cast<GetElementPtrInst>(&I)) { unsigned BitWidth = M.getDataLayout().getIndexSizeInBits(GEP->getPointerAddressSpace()); // Rewrite a constant GEP into a DIExpression. |