Review Request: Improved stack overflow handling for Zero
Christian.Thalinger at Sun.COM
Tue Apr 6 08:49:15 PDT 2010
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:
> 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.
6941224: Improved stack overflow handling for Zero
More information about the hotspot-compiler-dev