This is an archive of the discontinued LLVM Phabricator instance.

Differential D4707

Coverage: fix the missing output stream in recursive call
ClosedPublic

Authored by arphaman on Jul 29 2014, 9:11 AM.

Details

Reviewers
bob.wilson
arphaman
bogner
Summary

Fixes the missing output stream parameter in recursive call in CoverageMappingContext::dump

Diff Detail

Event Timeline

arphaman updated this revision to Diff 11984.Jul 29 2014, 9:11 AM
arphaman retitled this revision from to Coverage: fix the missing output stream in recursive call.
arphaman updated this object.
arphaman edited the test plan for this revision. (Show Details)
arphaman added reviewers: bogner, bob.wilson.
arphaman set the repository for this revision to rL LLVM.
arphaman added a subscriber: Unknown Object (MLST).
arphaman accepted this revision.Jul 29 2014, 1:07 PM
arphaman added a reviewer: arphaman.

Commited in r214206.

This revision is now accepted and ready to land.Jul 29 2014, 1:07 PM
arphaman closed this revision.Jul 29 2014, 1:07 PM

Revision Contents

PathSize
lib/
ProfileData/
4 lines

Diff 11984

lib/ProfileData/CoverageMapping.cpp

Context not available.
return; return;
const auto &E = Expressions[C.getExpressionID()]; const auto &E = Expressions[C.getExpressionID()];
OS << '('; OS << '(';
dump(E.LHS); dump(E.LHS, OS);
OS << (E.Kind == CounterExpression::Subtract ? " - " : " + "); OS << (E.Kind == CounterExpression::Subtract ? " - " : " + ");
dump(E.RHS); dump(E.RHS, OS);
OS << ')'; OS << ')';
break; break;
} }
Context not available.