[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