RFR: 8287794: Reverse*VNode::Identity problem [v2]
Jatin Bhateja
jbhateja at openjdk.org
Mon Jul 25 07:51:48 UTC 2022
On Mon, 25 Jul 2022 06:13:09 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:
>> No, since the flag Predicated_with_blend is set over nodes if operation itself is predicated, here both ReverseBytesV nodes are non-predicated ones and this check is part of identity routines of reverse* operations.
>
> Oh, right! What I mean is the case like:
>
> VectorBlend (ReverseBytesV X) (ReverseBytesV (ReverseBytesV X)) MASK ==> VectorBlend (ReverseBytesV X) X MASK
>
> which is the same case with `test_reversebytes_long_transform2` for non-predicated systems.
As already [discussed](https://github.com/openjdk/panama-vector/pull/182#discussion_r927419799), we can handle this as a separate PR along with other complimentary operations. This bug fix patch is fixing a specific issue.
>> CPU features list is populated during VM startup, this list is later ON queried by applyIfCPUFeature during IR validations, I do not think UseSVE is a valid flag for X86,
>>
>>
>> SPROMPT>java -XX:UseSVE=0
>> Unrecognized VM option 'UseSVE=0'
>> Error: Could not create the Java Virtual Machine.
>> Error: A fatal exception has occurred. Program will exit.
>> SPROMPT>lscpu
>> Architecture: x86_64
>> CPU op-mode(s): 32-bit, 64-bit
>> Byte Order: Little Endian
>> Address sizes: 46 bits physical, 48 bits virtual
>> CPU(s): 72
>> On-line CPU(s) list: 0-71
>> Thread(s) per core: 2
>> Core(s) per socket: 18
>> Socket(s): 2
>> NUMA node(s): 2
>> Vendor ID: GenuineIntel
>> CPU family: 6
>> Model: 85
>> Model name: Intel(R) Xeon(R) Gold 6139 CPU @ 2.30GHz
>
> Yes, it's invalid on x86. So maybe you could add the limitation to the "requires", but seems this could make the codes complex.
Correct.
-------------
PR: https://git.openjdk.org/jdk/pull/9623
More information about the hotspot-compiler-dev
mailing list