review (S) for 6857159: local schedule failed with checkcast of Thread.currentThread()

Tom Rodriguez Thomas.Rodriguez at Sun.COM
Tue Jul 21 14:31:06 PDT 2009

> Can you use is_immutable_load() instead of uses_immutable_memory()
> since it is load from immutable memory?

That's what I had originally and I changed it since it's technically  
not quite true.  The LoadKlass that reads from the cache isn't  
actually reading immutable memory since the cache can be updated,  
though never by generated code.  Neither name is really right.  Maybe  
it should just be disable_antidep_check with a comment explaining why?

> And change comment to include LoadRange:
> // node matches ideal 'LoadKlassNode' 'LoadNKlassNode'
> ---
> // node matches ideal 'LoadKlass' 'LoadNKlass' 'LoadRange'



> Thanks,
> Vladimir
> Tom Rodriguez wrote:

More information about the hotspot-compiler-dev mailing list