On Wed, 6 Sep 2023 18:34:29 GMT, Mandy Chung <mch...@openjdk.org> wrote:
> This reimplements > `sun.reflect.ReflectionFactory::newConstructorForSerialization` with method > handles. > > This API currently generates the bytecode which fails the verification > because `new C; invokespecial A()` where the given class `C` and invoke a > no-arg constructor of `C`'s first non-`Serializable` superclass `A` is not a > valid operation per the VM specification. VM special cases the classes > generated for reflection to skip verification for the constructors generated > for serialization and externalization. This change will allow such VM hack > to be removed. > > A `jdk.reflect.useOldSerializableConstructor` system property can be set to > use the old implementation in case if customers run into any compatibility > issue. I expect this change has very low compatibility risk. This system > property is undocumented and will be removed in a future release. This pull request has now been integrated. Changeset: 5cea53d3 Author: Mandy Chung <mch...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/5cea53d372744ddf1bedaae4667415e6525ef82f Stats: 151 lines in 10 files changed: 111 ins; 11 del; 29 mod 8315810: Reimplement sun.reflect.ReflectionFactory::newConstructorForSerialization with method handles Co-authored-by: Chen Liang <li...@openjdk.org> Reviewed-by: rriggs ------------- PR: https://git.openjdk.org/jdk/pull/15600
