RFR: 8286125: C2: "bad AD file" with PopulateIndex on x86_64 [v2]

[Tobias Hartmann]

On Tue, 10 May 2022 05:09:44 GMT, Pengfei Li <pli at openjdk.org> wrote:

> As in post loops, all packs are singleton, i.e., have_same_inputs is always true.
Hence, we also add a pack size check here to make post loop logic run
into this block. It's safe to let it go because post loop never needs
scalar replications of the loop IV - it never combines nodes in packs.

I don't understand why we need to execute that code for post loops. Could you elaborate why the `p->size() == 1` check is needed?

test/hotspot/jtreg/compiler/vectorization/TestReplicateLoopIV.java line 1:

> 1: 

Suggestion:

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

PR: https://git.openjdk.java.net/jdk/pull/8587