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

[Tom Rodriguez]

> 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'

Ok.

tom

>
> Thanks,
> Vladimir
>
> Tom Rodriguez wrote:
>> http://cr.openjdk.java.net/~never/6857159