[10] RFR(S): 8191362: [Graal] gc/g1/TestShrinkAuxiliaryData tests crash with "assert(check_klass_alignment(result)) failed: address not aligned"

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[10] RFR(S): 8191362: [Graal] gc/g1/TestShrinkAuxiliaryData tests crash with "assert(check_klass_alignment(result)) failed: address not aligned"

Tobias Hartmann-2
Hi,

please review the following patch:
https://bugs.openjdk.java.net/browse/JDK-8191362
http://cr.openjdk.java.net/~thartmann/8191362/webrev.03/

Graal should respect the ObjectAlignmentInBytes flag when computing array allocation sizes. This already has been
reviewed by Tom Rodriguez and Gilles Duboscq (see [GR-7679]) and was pushed to the Graal/jvmci repos.

Thanks,
Tobias
Reply | Threaded
Open this post in threaded view
|

Re: [10] RFR(S): 8191362: [Graal] gc/g1/TestShrinkAuxiliaryData tests crash with "assert(check_klass_alignment(result)) failed: address not aligned"

Vladimir Kozlov
Good.

thanks,
Vladimir K

On 1/9/18 10:41 AM, Tobias Hartmann wrote:

> Hi,
>
> please review the following patch:
> https://bugs.openjdk.java.net/browse/JDK-8191362
> http://cr.openjdk.java.net/~thartmann/8191362/webrev.03/
>
> Graal should respect the ObjectAlignmentInBytes flag when computing array allocation sizes. This already has been
> reviewed by Tom Rodriguez and Gilles Duboscq (see [GR-7679]) and was pushed to the Graal/jvmci repos.
>
> Thanks,
> Tobias
>
Reply | Threaded
Open this post in threaded view
|

Re: [10] RFR(S): 8191362: [Graal] gc/g1/TestShrinkAuxiliaryData tests crash with "assert(check_klass_alignment(result)) failed: address not aligned"

Tobias Hartmann-2
Thanks Vladimir.

Best regards,
Tobias

On 09.01.2018 20:51, Vladimir Kozlov wrote:

> Good.
>
> thanks,
> Vladimir K
>
> On 1/9/18 10:41 AM, Tobias Hartmann wrote:
>> Hi,
>>
>> please review the following patch:
>> https://bugs.openjdk.java.net/browse/JDK-8191362
>> http://cr.openjdk.java.net/~thartmann/8191362/webrev.03/
>>
>> Graal should respect the ObjectAlignmentInBytes flag when computing array allocation sizes. This already has been
>> reviewed by Tom Rodriguez and Gilles Duboscq (see [GR-7679]) and was pushed to the Graal/jvmci repos.
>>
>> Thanks,
>> Tobias
>>