[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