RFR: 8287794: Reverse*VNode::Identity problem [v4]

Xiaohong Gong xgong at openjdk.org
Thu Jul 28 07:24:34 UTC 2022


On Thu, 28 Jul 2022 07:04:44 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> Thanks for updating the tests! I ran the test in our internal testing system,  and unfortunately, this case will fail with "-XX:UseSVE=0" as expected. The reason is actually like what I said above. So to fix this, could you please:
>> 
>> 1. Limit the whole test on aarch64 os systems by adding "`@requires vm.cpu.features ~= ".*simd.*"`" before the test. And limit the single IR test by adding "`applyIf={"UseSVE", ">0"}`" .
>> 
>> OR
>> 
>> 2. Fix the IR framework, to make "applyIf" and "applyIfCPUFeature" can co-work with each other.
>> 
>> WDYT?
>
> I am not clear, if UseSVE=0 then why does "sve" feature getting populated during VM initialization?
> Entire handling of applyIfCPUFeature* is based on white box CPU feature API which queries the feature list populated during VM initialization, we are not directly queries the target features seen in /proc/cpuinfo.

Yeah, that's the difference from X86. `UseSVE` is a vm flag which lets user to choose whether use the sve feature or not. And the sve cpu feature is still there that will not been influenced by the VM flag.

-------------

PR: https://git.openjdk.org/jdk/pull/9623


More information about the hotspot-compiler-dev mailing list