RFR (S) 8145428: Optimize StringUTF16 compress/copy methods for C1

Roger Riggs Roger.Riggs at Oracle.com
Thu Dec 17 14:55:31 UTC 2015


Hi Alexsey,

The 'expected to run benchmarks' might the operative comment in the code.
'Common' knowledge sometimes isn't so common.

Roger


On 12/17/2015 2:54 AM, Aleksey Shipilev wrote:
> On 12/17/2015 02:34 AM, Ulf wrote:
>> I'm wondering why moving the increment operation to an extra line wound
>> enhance performance.
> Because C1 is very straightforward, and code movement like that is a
> poor man's instruction scheduling, that pads out the data dependency
> between index update and indexed access. I don't think it deserves a
> comment -- it is expected one will run the benchmarks when changing that
> code.
>
> Thanks,
> -Aleksey
>




More information about the core-libs-dev mailing list