[lworld+vector] RFR: 8314980: [lworld+vector] consider scalarization conditions during ciMultiField creation. [v3]

Jatin Bhateja jbhateja at openjdk.org
Thu Aug 31 05:36:50 UTC 2023


On Wed, 30 Aug 2023 01:53:10 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:

>> That is one which is being used by compilers. The transient one is there for ciInstanceKlass::populate_synthetic_multifields. Both c1 and C2 use ciBytecodeStream::get_field API which creates a ciField from field reference in constant pool. It does not take into account annotations over the field and hence populate synthetic fields operates over originally created field to create a fully populated ciMultifiled. I wanted to avoid adding a transient flag originally.
>
> Yes, I'm wondering whether we can use one like before, by just setting `_is_multifield_base = true` only when the multifields are vectorized.

Fixed the messy nit.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/918#discussion_r1311111993



More information about the valhalla-dev mailing list