This is an archive of the discontinued LLVM Phabricator instance.

[DebugInfo][EarlyCSE] Use the salvageDebugInfoOrMarkUndef(); NFC
ClosedPublic

Authored by djtodoro on Dec 5 2019, 6:02 AM.

Reviewers

aprantl
vsk

Commits

Summary

Use the newest API.

Lgtm.

This revision is now accepted and ready to land.Dec 5 2019, 8:41 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

llvm/

lib/

Transforms/

Scalar/

4 lines

Show First 20 Lines • Show All 901 Lines • ▼ Show 20 Lines	for (BasicBlock::iterator I = BB->begin(), E = BB->end(); I != E;) {

// Dead instructions should just be removed.		// Dead instructions should just be removed.
if (isInstructionTriviallyDead(Inst, &TLI)) {		if (isInstructionTriviallyDead(Inst, &TLI)) {
LLVM_DEBUG(dbgs() << "EarlyCSE DCE: " << *Inst << '\n');		LLVM_DEBUG(dbgs() << "EarlyCSE DCE: " << *Inst << '\n');
if (!DebugCounter::shouldExecute(CSECounter)) {		if (!DebugCounter::shouldExecute(CSECounter)) {
LLVM_DEBUG(dbgs() << "Skipping due to debug counter\n");		LLVM_DEBUG(dbgs() << "Skipping due to debug counter\n");
continue;		continue;
}		}
if (!salvageDebugInfo(*Inst))
replaceDbgUsesWithUndef(Inst);		salvageDebugInfoOrMarkUndef(*Inst);
removeMSSA(Inst);		removeMSSA(Inst);
Inst->eraseFromParent();		Inst->eraseFromParent();
Changed = true;		Changed = true;
++NumSimplify;		++NumSimplify;
continue;		continue;
}		}

// Skip assume intrinsics, they don't really have side effects (although		// Skip assume intrinsics, they don't really have side effects (although
▲ Show 20 Lines • Show All 508 Lines • Show Last 20 Lines