This is an archive of the discontinued LLVM Phabricator instance.

Differential D29906

Correct typo s/hosting/hoisting/
ClosedPublic

Authored by bcain on Feb 13 2017, 12:35 PM.

Details

Reviewers
anemet

Diff Detail

Repository
rL LLVM

Event Timeline

bcain created this revision.Feb 13 2017, 12:35 PM
anemet accepted this revision.Feb 13 2017, 1:38 PM

LGTM, thanks.

This revision is now accepted and ready to land.Feb 13 2017, 1:38 PM
bcain updated this revision to Diff 88382.Feb 14 2017, 8:36 AM

Included corresponding test cases.

anemet added a comment.Feb 14 2017, 8:50 AM

Still looks good!

bcain closed this revision.Feb 19 2017, 1:27 PM

Revision Contents

PathSize
lib/
Transforms/
Scalar/
2 lines

Diff 88245

lib/Transforms/Scalar/LICM.cpp

Context not available.
DEBUG(dbgs() << "LICM hoisting to " << Preheader->getName() << ": " << I DEBUG(dbgs() << "LICM hoisting to " << Preheader->getName() << ": " << I
<< "\n"); << "\n");
ORE->emit(OptimizationRemark(DEBUG_TYPE, "Hoisted", &I) ORE->emit(OptimizationRemark(DEBUG_TYPE, "Hoisted", &I)
<< "hosting " << ore::NV("Inst", &I)); << "hoisting " << ore::NV("Inst", &I));
// Metadata can be dependent on conditions we are hoisting above. // Metadata can be dependent on conditions we are hoisting above.
// Conservatively strip all metadata on the instruction unless we were // Conservatively strip all metadata on the instruction unless we were
Context not available.