[PATCH] D19002: [LazyValueInfo] Fix for a nasty compile-time problem with questions

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 26 19:34:10 PDT 2016


reames added a comment.

In http://reviews.llvm.org/D19002#413301, @llvm-commits wrote:

> Yes, I had already kicked off a build with your patches. Unfortunately it does not (at least no significantly) fix the issue and the test case is still running (for about 30 min now). My fix brings CT down to 2 min on an RA compiler.


Surprising.

Geoff, any chance you could share a test case?  I'd really like to understand why this is taking so long.

p.s. I think my earlier comment was unclear.  Assuming null pointer case in http://reviews.llvm.org/D18066 resolves your compile time problem, I plan on landing that one and not this one.  The effect should be roughly the same, but yours is a much deeper hack in the algorithm itself.


http://reviews.llvm.org/D19002





More information about the llvm-commits mailing list