RFR: 8265491: Math Signum optimization for x86 [v2]
Marcus G K Williams
github.com+168222+mgkwill at openjdk.java.net
Tue Apr 20 15:28:48 UTC 2021
On Tue, 20 Apr 2021 14:57:09 GMT, Jie Fu <jiefu at openjdk.org> wrote:
>> Marcus G K Williams has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Add os.arch=="i386" to signum jtreg
>>
>> Signed-off-by: Marcus G K Williams <marcus.williams at intel.com>
>
> test/hotspot/jtreg/compiler/intrinsics/math/TestSignumIntrinsic.java line 28:
>
>> 26: * @test
>> 27: * @summary Test compiler intrinsics for signum
>> 28: * @requires os.arch=="aarch64" | os.arch=="x86" | os.arch=="amd64" | os.arch=="x86_64"
>
> What about os.arch=="i386" ?
@DamonFool, I wasn't sure about `os.arch=="i386"`. I'm still getting my head around os.arch and how it is used in these tests.
I'll add it to the jtreg.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3581
More information about the hotspot-compiler-dev
mailing list