RFR(XXS): JDK-8196950: AARCH64 - Add VM flags presets for Cavium Thunder X2 CPU
dmitry.chuyko at bell-sw.com
Tue Nov 6 10:45:39 UTC 2018
On 11/2/18 6:20 PM, Andrew Dinn wrote:
> Hi Dmitry,
> On 02/11/18 14:01, Dmitry Chuyko wrote:
>> On 11/1/18 12:47 PM, Andrew Dinn wrote:
>>> On 31/10/18 17:40, Dmitry Chuyko wrote:
>> That's an option. Though "default" (false) and "false" are not exactly
>> the same, in the sense of explicitly expressing the difference in
>> benchmarks score, we'll have history for that.
> I am afraid don't know what that is supposed to mean. Can you try
> explaining it again, perhaps in more detail?
>> Webrev with updated patch:
> Well, the patch now looks good to me. However, I'm not sure if your
> comment means I need to revise that view. My belief is that:
> i) this is the better way to patch the code (because it removes
> unnecessary code)
> ii) the result will be functionally /identical/ to that obtained using
> the patch you initially suggested
> If your statement above is supposed to counter the second of those
> beliefs is wrong I'd be glad to be corrected.
Andrew, thanks for the review. I definitely agree with both statements
OK to push?
> Andrew Dinn
More information about the hotspot-compiler-dev