8214307 [lworld][C1] Access to flattened fields does not support nested flattened fields

[Ioi Lam]

BTW, looks like we have a similar issues here as well 
(ciInstanceKlass::compute_nonstatic_fields_impl):

http://hg.openjdk.java.net/valhalla/valhalla/file/448c8cd077c9/src/hotspot/share/ci/ciInstanceKlass.cpp#l494

It just expands nested flattened fields by one level.

Thanks

- Ioi


On 11/26/18 9:42 PM, Ioi Lam wrote:
> Looks good. I can verify that TestBasicFunctionality.test26 failed 
> without your patch, and passed with your patch.
>
> Thanks
>
> - Ioi
>
>
> On 11/26/18 1:34 PM, Frederic Parain wrote:
>> Small update to the C1 work:
>>
>> The initial code to handle flattened fields was missing the support 
>> for nested flattened fields.
>> The following patch fixes it.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~fparain/C1/webrev.02/index.html
>>
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8214307
>>
>> Fred
>>
>