DoubleAccumulator and RISC-V weirdness

Doug Lea dl at cs.oswego.edu
Wed Aug 24 19:34:02 UTC 2022


Thanks for finding and fixing this. Embarrassingly, we once received a 
bug report about an application also misusing identity, and closed it 
without checking if the same error appeared in tests.

On 8/24/22 14:15, Aleksey Shipilev wrote:
> On 8/24/22 19:33, Aleksey Shipilev wrote:
>> On 8/24/22 19:23, Aleksey Shipilev wrote:
>>> Thoughts? If you agree this is a bug, I can PR the fix :)
>>
>> Dang. The minute I hit the "Send" button I of course realized this is 
>> "identity" (!), so applying
>> the function to it was supposed to be neutral. It is then the user 
>> error to put in binary addition
>> as the function, for which "13.9" is not an identity.
>>
>> The test we looked at is the minimized version of 
>> ./test/jdk/java/util/concurrent/atomic/Serial.java:
>> https://github.com/openjdk/jdk/blob/568be58e8521e5e87baca1872ba8cc1941607bb7/test/jdk/java/util/concurrent/atomic/Serial.java#L66-L79
>>
>> ...which makes it a test bug?
>
> Here we go:
>   https://bugs.openjdk.org/browse/JDK-8292877
>   https://github.com/openjdk/jdk/pull/10002
>


More information about the riscv-port-dev mailing list