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
This commit is contained in:
Bjorn Pettersson 2018-06-19 08:41:34 +00:00
parent 11dc6a1442
commit 4347fa44b1

View File

@ -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();