RFR: 8304042: C2 SuperWord: schedule must remove packs with cyclic dependencies [v2]

[Emanuel Peter]

On Thu, 30 Mar 2023 16:29:10 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>>> Do you know if this affect any our existing vector tests?
>> 
>> @vladimir Thanks for the review.
>> Yes. I had a run where I assert if I find cycles. I ran it up to tier5 and stress testing. And the assert was never triggered, except in the two regression tests that I added (there it triggered a lot). So I think it really has no effect, except the extra runtime.
>
>> > Do you know if this affect any our existing vector tests?
>> 
>> @vladimir Thanks for the review. Yes. I had a run where I assert if I find cycles. I ran it up to tier5 and stress testing. And the assert was never triggered, except in the two regression tests that I added (there it triggered a lot). So I think it really has no effect, except the extra runtime.
> 
> Perfect! Thank you for doing it.

Thanks @vnkozlov @fg1417 @TobiHartmann for the reviews and suggestions!

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

PR Comment: https://git.openjdk.org/jdk/pull/13078#issuecomment-1496913196