RFR: 8289186: Support predicated vector load/store operations over X86 AVX2 targets. [v5]

Jatin Bhateja jbhateja at openjdk.org
Fri Jul 8 08:38:36 UTC 2022


> Hi All,
> 
> [JDK-8283667](https://bugs.openjdk.org/browse/JDK-8283667) added the support to handle masked loads on non-predicated targets by blending the loaded contents with zero vector iff unmasked portion of load does not span beyond array bounds.
> 
> X86 AVX2 offers direct predicated vector loads/store instruction for non-sub word type.
> 
> This patch adds the efficient backend implementation for predicated memory operations over int/long/float/double vectors.
> 
> Please find below the JMH micro stats with and without patch.
> 
> 
> 
> System : Intel(R) Xeon(R) Platinum 8280 CPU @ 2.70GHz [28C 2S Cascadelake Server]
> 
> Baseline:
> Benchmark                                          (inSize)  (outSize)   Mode  Cnt    Score   Error   Units
> LoadMaskedIOOBEBenchmark.byteLoadArrayMaskIOOBE        1026       1152  thrpt    2  712.218          ops/ms
> LoadMaskedIOOBEBenchmark.doubleLoadArrayMaskIOOBE      1026       1152  thrpt    2  156.912          ops/ms
> LoadMaskedIOOBEBenchmark.floatLoadArrayMaskIOOBE       1026       1152  thrpt    2  255.814          ops/ms
> LoadMaskedIOOBEBenchmark.intLoadArrayMaskIOOBE         1026       1152  thrpt    2  267.688          ops/ms
> LoadMaskedIOOBEBenchmark.longLoadArrayMaskIOOBE        1026       1152  thrpt    2  140.957          ops/ms
> LoadMaskedIOOBEBenchmark.shortLoadArrayMaskIOOBE       1026       1152  thrpt    2  474.009          ops/ms
> 
> 
> With Opt:
> Benchmark                                          (inSize)  (outSize)   Mode  Cnt     Score   Error   Units
> LoadMaskedIOOBEBenchmark.byteLoadArrayMaskIOOBE        1026       1152  thrpt    2   742.781          ops/ms
> LoadMaskedIOOBEBenchmark.doubleLoadArrayMaskIOOBE      1026       1152  thrpt    2  1241.021          ops/ms
> LoadMaskedIOOBEBenchmark.floatLoadArrayMaskIOOBE       1026       1152  thrpt    2  2333.311          ops/ms
> LoadMaskedIOOBEBenchmark.intLoadArrayMaskIOOBE         1026       1152  thrpt    2  3258.754          ops/ms
> LoadMaskedIOOBEBenchmark.longLoadArrayMaskIOOBE        1026       1152  thrpt    2  1757.192          ops/ms
> LoadMaskedIOOBEBenchmark.shortLoadArrayMaskIOOBE       1026       1152  thrpt    2   472.590          ops/ms
> 
> 
> Predicated memory operation over sub-word type will be handled in a subsequent patch. 
> 
> Kindly review and share your feedback.
> 
> Best Regards,
> Jatin

Jatin Bhateja has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains five commits:

 - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8289186
 - 8289186: jcheck failure
 - 8289186: Review comments resolved.
 - 8289186: Review comments resolved.
 - 8289186: Support predicated vector load/store operations over X86 AVX2 targets.

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

Changes: https://git.openjdk.org/jdk/pull/9324/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9324&range=04
  Stats: 330 lines in 8 files changed: 290 ins; 39 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/9324.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9324/head:pull/9324

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


More information about the hotspot-compiler-dev mailing list