Request for reviews (S): 7160610: Unknown Native Code compilation issue

[Vladimir Kozlov]

On 4/11/12 8:49 PM, Tom Rodriguez wrote:
>
> On Apr 11, 2012, at 6:52 PM, Vladimir Kozlov wrote:
>
>> http://cr.openjdk.java.net/~kvn/7160610/webrev
>>
>> 7160610: Unknown Native Code compilation issue
>>
>> When constructing input vector use type of vector's operation which use it since element's sizes should match. In the bug case byte value is used for initialization of int[] array. The store vector is Store2I and input's type (byte) was used to construct init vector but there is no Replicate2B. Even if Replicate2B existed it will be wrong since it initialize only lowest 2 bytes.
>
> Why does in_bb(opd) ever affect which type to use?


If you ask why we had this condition, I don't know. I also think it is bogus.

If you are asking about next line in my fix I will change it. After sending webrev I realized that the assert which use 
it is useless for the reason you asked:

    DEBUG_ONLY( const Type* opd_t = velt_type(!in_bb(opd) ? p0 : opd); )

     assert(opd_t == velt_type(!in_bb(in) ? pi : in), "all same type");

I will change it to next to verify that all packed input's has the same type (they should, since they come from loop 
unrolling/node cloning):

    DEBUG_ONLY( const Type* opd_t = opd->bottom_type();

     assert(opd_t == in->bottom_type(), "all same type");

Vladimir

>
> tom
>
>>
>> Thanks,
>> Vladimir
>>
>