[lworld+vector] RFR: 8314980: [lworld+vector] consider scalarization conditions during ciMultiField creation. [v7]
Jatin Bhateja
jbhateja at openjdk.org
Tue Sep 5 11:17:06 UTC 2023
On Tue, 5 Sep 2023 00:34:25 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:
>> Patch adds a new API _ciEnv::is_multifield_scalarized_, to scalarize multifield (ciField[s]) in case target vector cannot accommodate multifield bundle size, else it creates a hierarchical structure ciMultiField and expose entire multifield bundle as one field to C2 compiler.
>>
>> This cleans up special handling done in C2 compiler, ci field query APIs and object reconstruction handling at SafePoint.
>>
>> Please review and share your feedback.
>>
>> Best Regards,
>> Jatin
>
> Jatin Bhateja has updated the pull request incrementally with two additional commits since the last revision:
>
> - Minor cleanup
> - Check to ensure C1 compiler only build is successful.
There is one intermittent failure in following test on KNL
Double64VectorTests.unsliceBinaryDouble64VectorTestsBinary(double[-i * 5], double[cornerCaseValue(i)]): failure
But, we have also seen similar assertion failures in other shuffle / mask tests. Plan to addressed in subsequent patch.
-------------
PR Comment: https://git.openjdk.org/valhalla/pull/918#issuecomment-1706418952
More information about the valhalla-dev
mailing list