[PATCH] Coverage: fix the missing output stream in recursive call
Justin Bogner
mail at justinbogner.com
Tue Jul 29 12:20:56 PDT 2014
Alex Lorenz <arphaman at gmail.com> writes:
> Fixes the missing output stream parameter in recursive call in
> CoverageMappingContext::dump
LGTM. For obvious fixes like this one you can just commit and it'll get
looked at post-commit.
> Index: lib/ProfileData/CoverageMapping.cpp
> ===================================================================
> --- lib/ProfileData/CoverageMapping.cpp
> +++ lib/ProfileData/CoverageMapping.cpp
> @@ -101,9 +101,9 @@
> return;
> const auto &E = Expressions[C.getExpressionID()];
> OS << '(';
> - dump(E.LHS);
> + dump(E.LHS, OS);
> OS << (E.Kind == CounterExpression::Subtract ? " - " : " + ");
> - dump(E.RHS);
> + dump(E.RHS, OS);
> OS << ')';
> break;
> }
More information about the llvm-commits
mailing list