[lworld] RFR: 8281323: [lworld] Unnecessary entries in Preload attribute

[Srikanth Adayapalam]

On Wed, 23 Feb 2022 10:33:30 GMT, Aggelos Biboudis <duke at openjdk.java.net> wrote:

> This is fully covered by `requiresPreload` in `Type`. Actually more strict, because of the `if (this.isReferenceProjection()) return true;` condition in the previous method. Is my understanding correct?

Yes, you are correct. The check

if (this.isReferenceProjection()) return true;

covers the requirement for https://bugs.openjdk.java.net/browse/JDK-8280942

The check

return this.isValueClass() && !this.isPrimitiveClass();

works for "pure" B2 classes. (every B3 is also a B2)

So we are saying in effect preload is required for pure B2 classes or for a B3 class whose reference projection is used (in field/method descriptors and/or local types)

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

PR: https://git.openjdk.java.net/valhalla/pull/656