[aarch64-port-dev ] -XX:InlineSmallCode=1500

[Andrew Dinn]

On 24/07/17 15:46, Andrew Haley wrote:
> On 24/07/17 15:37, Andrew Dinn wrote:
>> Interestingly, there was also a comment in hotspot test
>> compiler/c2/Test6910605_2.java saying
>>
>>   "Added InlineSmallCode=2000 to guaranty [sic] inlining of
>> StringBuilder::append() to allow scalar replace StringBuilder object."
>>
>> Is this benefit anywhere near the vicinity of the improvement arrived at
>> in Doug's benchmark test?
> 
> Yes.  This explains another mystery: we only saw the slowdown with
> Doug's test when using -XX:-TieredCompilation.  So, it looks like this
> is already fixed for everything, as long as tiered compilation is
> enabled.  I guess it doesn't much matter whether we change it or not

Not much. It would probably be more rational to align the two settings
so AArch64 is consistently in the SPARC ballpark ... or we could just
leave it as a puzzle for future maintainers.

regards,


Andrew Dinn
-----------