RFR: JDK-8134157: adlc fails to compile with SS12u4

[Magnus Ihse Bursie]

On 2015-08-27 10:51, Erik Joelsson wrote:
> Hello,
>
> You are probably right regarding the future versions of the Solaris 
> Studio compiler. Here is a new webrev with the check being >= 513 
> (which is indeed 12.4).
>
> http://cr.openjdk.java.net/~erikj/8134157/webrev.02/
>
> I still only apply it to adlc as that's where +w is active. I doubt +w 
> will be activated for the rest of hotspot anytime soon as the amount 
> of warnings triggered is quite substantial.
Looks good to me.

/Magnus
>
> /Erik
>
> On 2015-08-26 20:37, Kim Barrett wrote:
>> On Aug 26, 2015, at 7:44 AM, Erik Joelsson <erik.joelsson at oracle.com> 
>> wrote:
>>> Further investigation revealed that this "error" was actually just a 
>>> warning being upgraded to error by -errwarn. The warning is only 
>>> activated by +w, which is turned on for adlc but not for the rest of 
>>> the hotspot build. Based on this, I have revised my proposal to just 
>>> disabling this warning for adlc and the particular compiler version.
>>>
>>> Webrev: http://cr.openjdk.java.net/~erikj/8134157/webrev.01/
>> I think the new approach is good, e.g. disable the specific warning.
>> However, only doing so for this one compiler rev (I assume 513
>> corresponds to SS12u4?) seems like a mistake to me. I don't expect the
>> warning to go away with future compiler versions.
>>
>>
>