|
From: Julian S. <js...@ac...> - 2012-02-08 08:12:08
|
> I don't think that assertion can ever be true. Hey, but if you're not > sure, then it should definitely be there :) True :) > > we ought to put in a simple recursion depth check that stops it going > > nuts in the worst case. > > Yeah, we should do that.. I was planning to use a fixed depth across all > iropt levels. Fine. > I want to do some more runs for performance and also to see what this > buys us in terms of saved instructions. Clearly, it won't be earth > shattering, but I'm a still curious... I think it's likely to be a net loss -- the extra jitting cost is almost never outweighed by the improvements to generated code. For me, the win in this is that it makes it possible to get rid of some Memcheck false positives and hence make that tool more reliable. J |