Request for reviews (S): 6889656: assert(lo_lrg->lo_degree() || !lo_no_simplify,"Live range was lo-degree before coalesce
Thomas.Rodriguez at Sun.COM
Fri Oct 9 09:35:43 PDT 2009
On Oct 9, 2009, at 7:44 AM, Vladimir Kozlov wrote:
> Fixed 6889656: assert(lo_lrg->lo_degree() || !lo_no_simplify,"Live
> range was lo-degree before coalesce
> Because of initial loop's "i" value change in 6875959 fix
> the first _hi_degree lrg was not checked for lo_degree().
> Restore the original code: uint i = _hi_degree.
I missed that you changed the bounds in the original review.
> I also cached calculation of live ranges.
I don't really see the motivation for this and I'd prefer that this
fix to a fix just contained the fix and not other stuff, particularly
since the last bug was caused by "optimizing" the original code.
> Reviewed by:
> Fix verified (y/n): y, bug's test
> Other testing:
More information about the hotspot-compiler-dev