From 4347fa44b172ef5dae306084dd5fb8f95d95d3ad Mon Sep 17 00:00:00 2001 From: Bjorn Pettersson Date: Tue, 19 Jun 2018 08:41:34 +0000 Subject: [PATCH] Remove valueCoversEntireFragment asserts in ConvertDebugDeclareToDebugValue This is a fixup for r334830 causing problems in polly-aosp buildbot. Focus in r334830 was to fix a problem seen with ConvertDebugDeclareToDebugValue involving store instructions. It also added some asserts to find out of similar problems existed for the ConvertDebugDeclareToDebugValue functions involving load and phi instructions. One of those asserts seems to blow in the polly-aosp buildbot, so I'll revert the asserts while debugging. llvm-svn: 335031 --- lib/Transforms/Utils/Local.cpp | 6 ------ 1 file changed, 6 deletions(-) diff --git a/lib/Transforms/Utils/Local.cpp b/lib/Transforms/Utils/Local.cpp index 3d976005cba..3a43025cb83 100644 --- a/lib/Transforms/Utils/Local.cpp +++ b/lib/Transforms/Utils/Local.cpp @@ -1313,9 +1313,6 @@ void llvm::ConvertDebugDeclareToDebugValue(DbgInfoIntrinsic *DII, if (LdStHasDebugValue(DIVar, DIExpr, LI)) return; - assert(valueCoversEntireFragment(LI->getType(), DII) && - "Load is not loading the full variable fragment."); - // We are now tracking the loaded value instead of the address. In the // future if multi-location support is added to the IR, it might be // preferable to keep tracking both the loaded value and the original @@ -1336,9 +1333,6 @@ void llvm::ConvertDebugDeclareToDebugValue(DbgInfoIntrinsic *DII, if (PhiHasDebugValue(DIVar, DIExpr, APN)) return; - assert(valueCoversEntireFragment(APN->getType(), DII) && - "PHI node is not describing the full variable."); - BasicBlock *BB = APN->getParent(); auto InsertionPt = BB->getFirstInsertionPt();