RFR: 8232084: HotSpot build failed with GCC 9.2.1

Yasumasa Suenaga suenaga at oss.nttdata.com
Thu Oct 17 00:32:18 UTC 2019


Hi Serguei,

On 2019/10/17 9:07, serguei.spitsyn at oracle.com wrote:
> Hi Yasumasa,
> 
> It looks good.
> One tip:
> 
> + // This code would be warned as "stringop-truncation" by GCC 8 or later.
> +PRAGMA_DIAG_PUSH
> +PRAGMA_STRINGOP_TRUNCATION_IGNORED
>         strncpy(buf, str, len);
> +PRAGMA_DIAG_POP
> 
> 
> I'd suggest to place the comment before the line with strncopy().

Ok, I will move the comment before push.


Yasumasa


> Not  subject for re-review.
> 
> Thanks,
> Serguei
> 
> 
> On 10/16/19 16:25, Yasumasa Suenaga wrote:
>> (Re-send email because I could not send original email to serviceability-dev and hotspot-compiler-dev)
>>
>> Hi,
>>
>> We discussed the fix for JDK-8232084 in [1], and I think we should fix it with #pragma.
>> I uploaded new webrev. Could you review it?
>>
>>   JBS: https://bugs.openjdk.java.net/browse/JDK-8232084
>>   webrev: http://cr.openjdk.java.net/~ysuenaga/JDK-8232084/webrev.03/
>>
>> This change has passed the tests on submit repo (mach5-one-ysuenaga-JDK-8232084-1-20191016-1534-5969882).
>>
>>
>> Thanks,
>>
>> Yasumasa
>>
>>
>> [1] https://mail.openjdk.java.net/pipermail/serviceability-dev/2019-October/029547.html
> 


More information about the serviceability-dev mailing list