[EXTERNAL] Re: RFR(M): 8248190: PPC: Enable Power10 system and use new byte-reverse instructions
Thomas Schatzl
thomas.schatzl at oracle.com
Fri Aug 21 13:45:17 UTC 2020
Hi,
On 21.08.20 15:37, joserz at linux.ibm.com wrote:
> Hello!
>
> On Fri, Aug 21, 2020 at 10:04:38AM +0200, Thomas Schatzl wrote:
>> Hi,
>>
>> On 21.08.20 04:33, Michihiro Horie wrote:
>>>
>>> Hi Jose,
>>>
>>> One thing I noticed is a misaligned backslash in globals_ppc.hpp.
>>> Otherwise, the change looks good!
>>>
>>> /* special instructions */
>>> \
>>> + product(bool, UseByteReverseInstructions, false,
>>> \
>>
>> Fwiw, for adding product options, you must go through the CSR process. Maybe
>> there is an exception for platform specific ones?
>
> I didn't find any exception for platform specific options. But, "experimental" options
> don't need such CSR process and, to be honest, experimental seems more appropriate here.
> What do you think?
>
> Thank you for your review! :)
Just a fly-by. It's up to you :) - just that product options need to be
announced to the world.
I kind of agree that experimental seems more appropriate. You can always
"upgrade" it later.
Thomas
More information about the hotspot-compiler-dev
mailing list