Integrated: 8292443: Weak CAS VarHandle/Unsafe tests should test always-failing cases
Aleksey Shipilev
shade at openjdk.org
Wed Aug 17 18:17:28 UTC 2022
On Tue, 16 Aug 2022 15:06:50 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> I noticed that current VarHandle/Unsafe test cases do not test the always-failing cases. Strong CASes do test both always-success and always-failing scenarios, while weak CASes only test spurious-failure tests. We should be also testing always-failing cases for these. Weak CAS can spuriously fail, but it cannot spuriously succeed.
>
> The PR looks large, but the actual changes are in `*.template` files, and the rest is generated from them.
>
> Additional testing:
> - [x] Linux x86_64 fastdebug `java/lang/invoke/VarHandles` and `compiler/unsafe` tests
> - [x] Linux x86_32 fastdebug `java/lang/invoke/VarHandles` and `compiler/unsafe` tests
> - [x] Linux AArch64 fastdebug `java/lang/invoke/VarHandles` and `compiler/unsafe` tests
This pull request has now been integrated.
Changeset: e8bc8795
Author: Aleksey Shipilev <shade at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/e8bc87956abc92851de8694c56a78f6ecc546cbd
Stats: 2926 lines in 46 files changed: 2240 ins; 0 del; 686 mod
8292443: Weak CAS VarHandle/Unsafe tests should test always-failing cases
Reviewed-by: psandoz
-------------
PR: https://git.openjdk.org/jdk/pull/9892
More information about the hotspot-compiler-dev
mailing list