Integrated: 8259398: Super word not applied to a loop with byteArrayViewVarHandle
Vladimir Kozlov
kvn at openjdk.java.net
Mon Feb 1 15:51:47 UTC 2021
On Fri, 29 Jan 2021 17:40:39 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:
> Address expressing in this case has CastII which is not range check related.
> I think it is safe to skip any CastII nodes (similar to ConvI2L nodes) when parsing address for vectors - vectors will be constructed only if the same loop's variable and invariant are used for all memory operations regardless casts. Also vectors address depends on loop's variable so they will not be moved outside loop.
> In 32-bit VM there is no ConvI2L nodes so I moved CastII checks from under ConvI2L check.
>
> New regression case added to TestBufferVectorization.java test.
>
> Testing hs-tier1-7, RenaissanceStressTest. Run all vectorizing tests locally to make sure no regression.
This pull request has now been integrated.
Changeset: 02d586e1
Author: Vladimir Kozlov <kvn at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/02d586e1
Stats: 57 lines in 2 files changed: 41 ins; 6 del; 10 mod
8259398: Super word not applied to a loop with byteArrayViewVarHandle
Reviewed-by: vlivanov, thartmann, chagedorn
-------------
PR: https://git.openjdk.java.net/jdk/pull/2317
More information about the hotspot-compiler-dev
mailing list