RFR: 8222029: Optimize Math.floorMod

Joe Darcy joe.darcy at oracle.com
Tue Apr 9 16:02:53 UTC 2019


Basically I'm inquiring about whether the existing tests provide at 
least as good code coverage on the new implementation as the old one. As 
it is a relatively simple method, perhaps it there is full coverage 
before and after. However, at times changing the implementation requires 
updates to the tests to includes different cases to check and I wanted 
to make sure that was looked at here.

Thanks,

-Joe

On 4/9/2019 5:32 AM, Claes Redestad wrote:
> I think those tests cover all interesting corner cases, so the only way
> I see it can be improved is to make it more exhaustive (say generate a
> large random sample of tests every run). Do you feel that is needed?
>
> /Claes
>
> On 2019-04-09 01:35, Joseph D. Darcy wrote:
>> Should any additional cases be added to 
>> test/jdk/java/lang/Math/DivModTests.java to cover the new 
>> implementation?
>>
>> Thanks,
>>
>> -Joe
>>
>> On 4/5/2019 10:21 AM, Claes Redestad wrote:
>>>
>>>
>>> On 2019-04-05 17:41, Andrew Haley wrote:
>>>> On 4/5/19 2:44 PM, Claes Redestad wrote:
>>>>> Testing: tier1-2, all Math tests run locally, -prof perfasm 
>>>>> verification
>>>>> on the provided microbenchmark.
>>>>
>>>> Looks good.
>>>
>>> Thanks!
>>>
>>>>
>>>> I've kicked the tyres on AArch64, and it looks like a useful 
>>>> optimization. The
>>>> gains when the divisor is constant (a common case) are modest but 
>>>> worthwhile.
>>>>
>>>
>>> Thanks for trying it out and glad to hear it helps on AArch64 as well.
>>>
>>> /Claes
>>


More information about the core-libs-dev mailing list