Request for reviews (S): 7200233: C2: can't use expand rules for vector instruction rules

[Christian Thalinger]

On Sep 24, 2012, at 7:04 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:

> Christian Thalinger wrote:
>> On Sep 21, 2012, at 4:43 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>>> http://cr.openjdk.java.net/~kvn/7200233/webrev
>>> 
>>> Added missed _bottom_type set in ArchDesc::defineExpand().
>>> 
>>> Added missed vector nodes in MatchRule::is_vector() and in vmStructs.
>> Looks good.  But wouldn't it be better to have some kind of array with all the vector names in it for MatchRule::is_vector?
> 
> Done:
> 
> http://cr.openjdk.java.net/~kvn/7200233/webrev.01

Much better.  Thanks.  -- Chris

> 
> Thanks,
> Vladimir
> 
>> -- Chris
>>> Thanks,
>>> Vladimir