RFR: 8222029: Optimize Math.floorMod

Claes Redestad claes.redestad at oracle.com
Tue Apr 9 19:06:38 UTC 2019


In my cursory analysis of the test the interesting corner cases are
covered and I have no reason to believe we'd see spurious errors
elsewhere. I ran an adhoc semi-exhaustive test comparing results
of the new version with the old and got an all-pass.

/Claes

On 2019-04-09 18:02, Joe Darcy wrote:
> 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