RFR: 8346664: C2: Optimize mask check with constant offset [v12]

Emanuel Peter epeter at openjdk.org
Fri Jan 31 06:33:53 UTC 2025


On Fri, 31 Jan 2025 06:28:03 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Matthias Ernst has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   "should never vectorize" only holds for long[] input.
>
> test/hotspot/jtreg/compiler/loopopts/superword/TestEquivalentInvariants.java line 880:
> 
>> 878:         applyIfCPUFeatureOr = {"sse4.1", "true", "asimd", "true"})
>> 879:     // Would be nice if it vectorized.
>> 880:     // Fails because of control flow. Somehow the "offsetPlain" check (checks for alignment) is not folded away.
> 
> Suggestion:
> 
>     // With AlignVector (strict alignment requirements): we cannot prove that the invariants are alignable -> no vectorization.

Check where else this applies too now ;)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22856#discussion_r1936743851


More information about the hotspot-compiler-dev mailing list