On Sun, 21 Apr 2024 13:40:00 GMT, Jatin Bhateja <[email protected]> wrote:
>> Hi All, >> >> This patch optimizes sub-word gather operation for x86 targets with AVX2 and >> AVX512 features. >> >> Following is the summary of changes:- >> >> 1) Intrinsify sub-word gather using hybrid algorithm which initially >> partially unrolls scalar loop to accumulates values from gather indices into >> a quadword(64bit) slice followed by vector permutation to place the slice >> into appropriate vector lanes, it prevents code bloating and generates >> compact JIT sequence. This coupled with savings from expansive array >> allocation in existing java implementation translates into significant >> performance of 1.5-10x gains with included micro. >> >>  >> >> >> 2) Patch was also compared against modified java fallback implementation by >> replacing temporary array allocation with zero initialized vector and a >> scalar loops which inserts gathered values into vector. But, vector insert >> operation in higher vector lanes is a three step process which first >> extracts the upper vector 128 bit lane, updates it with gather subword value >> and then inserts the lane back to its original position. This makes inserts >> into higher order lanes costly w.r.t to proposed solution. In addition >> generated JIT code for modified fallback implementation was very bulky. This >> may impact in-lining decisions into caller contexts. >> >> 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 16 commits: > > - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8318650 > - Review resolutions. > - Review comment resolutions. > - Review comments resolutions > - Review comments resolutions. > - Review comments resolutions. > - Generalizing masked sub-gather support. > - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8318650 > - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8318650 > - Fix incorrect comment > - ... and 6 more: https://git.openjdk.org/jdk/compare/6d569961...b24cc5cd The comments by XiaohongGong were removed by the bot, but he has signed the OCA and is active elsewhere. Let's see if the bot revives the comments if I do this: ------------- PR Comment: https://git.openjdk.org/jdk/pull/16354#issuecomment-4600781371
