RFR: 8298935: fix cyclic dependency bug in create_pack logic in SuperWord::find_adjacent_refs [v9]

Emanuel Peter epeter at openjdk.org
Thu Mar 2 16:00:44 UTC 2023


On Thu, 2 Mar 2023 07:44:16 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>>> @jatin-bhateja I now have a first version out of the `test/hotspot/jtreg/compiler/loopopts/superword/TestDependencyOffsets.java`. It seems to work for `sse4.1 ... avx512`. I'm now testing it for `asimd`. And then I will proceed to add the features for `-XX:+AlignVector`, with the modulo check.
>> 
>> I am seeing lots of IR violations with UseSSE=4.
>
> I realized I have lots of negative IR rules that check that we do NOT vectorize if I expect cyclic dependency. But these negative rules are difficult, there may always be some other factor that leads to shorter vector sizes than what I expect. And then it vectorizes, and does not encounter a cyclic dependency. So I will have to remove all these negative IR rules.
> 
> @jatin-bhateja was there any positive IR rule that failed? One that did expect vectorization, but it did not in fact vectorize?

I now removed all such negative IR rules.

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

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


More information about the hotspot-compiler-dev mailing list