Request for reviews (S): 6877254: Server vm crashes with "no branches off of store slice" when run with CMS and UseSuperWord (default)
Vladimir Kozlov
Vladimir.Kozlov at Sun.COM
Tue Sep 8 18:30:41 PDT 2009
I have concerns about StoreCMNode::Ideal(aidx) implementation.
The result of memory_at() is never the same mergemem node.
I either mergemem->base_memory() or mergemem->in(aidx)
so your check will be always true.
Also mergemem nodes could be chained when the result of memory_at()
is also mergemem.
You can look on methods in memnode.cpp which search through
mergemem for stores: step_through_mergemem(), find_previous_store().
Thanks,
Vladimir
Changpeng Fang wrote:
> Need additional reviews (Tom (never) proposed the implementation, so I
> assume he is oked )!
> Comments are in the webrev.
>
>
> On 09/01/09 11:07, Changpeng Fang wrote:
>> http://cr.openjdk.java.net/~cfang/6877254/webrev.00/
>>
>> Thanks,
>>
>> Changpeng
>
>
> Thanks,
>
> Changpeng
More information about the hotspot-compiler-dev
mailing list