[llvm] r262103 - [sancov] do not instrument nodes that are full pre-dominators

[Mike Aizatsky via llvm-commits]

Renato,

I did. And I thought I fixed it in http://reviews.llvm.org/rL262111 (as soon
as I got breakage notification).

There's something wrong with those architectures. There are no problems on
others. I'll take better look on Monday.

On Sat, Feb 27, 2016 at 6:25 AM Renato Golin <renato.golin at linaro.org>
wrote:

> On 27 February 2016 at 02:10, Mike Aizatsky via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
> > Author: aizatsky
> > Date: Fri Feb 26 20:10:27 2016
> > New Revision: 262103
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=262103&view=rev
> > Log:
> > [sancov] do not instrument nodes that are full pre-dominators
> >
> > Summary:
> > Without tree pruning clang has 2,667,552 points.
> > Wiht only dominators pruning: 1,515,586.
> > With both dominators & predominators pruning: 1,340,534.
> >
> > Differential Revision: http://reviews.llvm.org/D17671
>
> I've just reverted this patch in r262139, as it broke all ARM and
> AArch64 buildbots. Please, pay attention to the bots and contact the
> owner or revert on the first sign of trouble.
>
> cheers,
> --renato
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160227/d2943b13/attachment.html>