[llvm-commits] [llvm] r172363 - in /llvm/trunk: include/llvm/Analysis/MemoryBuiltins.h lib/Analysis/MemoryBuiltins.cpp

[Joerg Sonnenberger]

On Sun, Jan 13, 2013 at 10:52:10AM -0800, Chandler Carruth wrote:
> On Sun, Jan 13, 2013 at 10:02 AM, Nuno Lopes <nunoplopes at sapo.pt> wrote:
> 
> > Author: nlopes
> > Date: Sun Jan 13 12:02:57 2013
> > New Revision: 172363
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=172363&view=rev
> > Log:
> > fix compile-time regression report by Joerg Sonnenberger:
> > cache result of Size/OffsetVisitor to speedup analysis of PHI nodes
> >
> 
> I suspect this wasn't a performance issue so much as a termination issue.
> The test case Joerg had inf-looped for me, and if I removed the 'erase'
> from the set, completed very rapidly.

The reduced test case did finish, so it might really have just been a
case of extreme slow behavior. Doing a clean build of the original
environment now to see if it is fixed.

Joerg