From 4fe42214e24227906c2d2be723465b8bef063ee8 Mon Sep 17 00:00:00 2001 From: Wolfgang Pieb Date: Mon, 15 Apr 2019 17:36:29 +0000 Subject: [DEBUGINFO] Prevent Instcombine from dropping debuginfo when removing zexts Zexts can be treated like no-op casts when it comes to assessing whether their removal affects debug info. Reviewer: aprantl Differential Revision: https://reviews.llvm.org/D60641 llvm-svn: 358431 --- llvm/lib/Transforms/Utils/Local.cpp | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'llvm/lib/Transforms/Utils/Local.cpp') diff --git a/llvm/lib/Transforms/Utils/Local.cpp b/llvm/lib/Transforms/Utils/Local.cpp index 3110322b314..65d5928e99c 100644 --- a/llvm/lib/Transforms/Utils/Local.cpp +++ b/llvm/lib/Transforms/Utils/Local.cpp @@ -1666,11 +1666,10 @@ DIExpression *llvm::salvageDebugInfoImpl(Instruction &I, }; if (auto *CI = dyn_cast(&I)) { - if (!CI->isNoopCast(DL)) - return nullptr; - - // No-op casts are irrelevant for debug info. - return SrcDIExpr; + // No-op casts and zexts are irrelevant for debug info. + if (CI->isNoopCast(DL) || isa(&I)) + return SrcDIExpr; + return nullptr; } else if (auto *GEP = dyn_cast(&I)) { unsigned BitWidth = M.getDataLayout().getIndexSizeInBits(GEP->getPointerAddressSpace()); -- cgit v1.2.3