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