Review Request: Improved stack overflow handling for Zero

[Christian Thalinger]

On Tue, 2010-04-06 at 16:09 +0100, Gary Benson wrote:
> Hi all,
> 
> I recently started adding stack overflow checking to Shark,
> which brought to light a bunch of deficiencies in Zero's
> stack overflow code.  Zero had this really odd, look-ahead
> overflow handling which was pretty strange, easily fooled,
> and in retrospect totally unnecessary.  I'm pretty sure
> whoever wrote it was an idiot.
> 
> This webrev removes the old overflow detection and replaces
> it with something more straightforward and complete:
> 
>   http://cr.openjdk.java.net/~gbenson/zero-stack-overflow-checks/
> 
> Note that a number of calls to Unimplemented have been removed.
> It shouldn't be possible to have an unhandled stack overflow
> error with the new code.
> 
> As a bonus, the new code is mildly faster (about 2%) due to
> the checks being inlined.
> 
> I don't have a bug id for this.

I created:

6941224: Improved stack overflow handling for Zero

-- Christian