[llvm] r262103 - [sancov] do not instrument nodes that are full pre-dominators
Mike Aizatsky via llvm-commits
llvm-commits at lists.llvm.org
Sat Feb 27 09:00:42 PST 2016
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>
> 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.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits