On Fri, 8 Mar 2024 06:51:21 GMT, Calvin Cheung <cche...@openjdk.org> wrote:
>> To avoid the CDS dump error message, a fix is during dumping a classlist, >> check if an invoker can be archived. >> If not, don't write the invoker info into the classlist, i.e. don't call >> `logLambdaFormInvoker()`. While generating holder classes (in >> `generateHolderClasses()`), don't add the `MethodType` to the `invokerTypes` >> if will fail the check in the `build()` method which would result in a >> `RuntimeException`. >> >> Also updated the `MethodHandlesInvokersTest.java` under >> `appcds/methodHandles` and `appcds/dynamicArchive/methodHandles` to check >> that the "Failed to generate LambdaForm holder classes" error is not in the >> output; >> >> Passed tiers 1 - 3 testing. > > Calvin Cheung has updated the pull request incrementally with one additional > commit since the last revision: > > @iklam comments and copyright update LGTM! Please don't forget to address Ioi's nit before integration, thanks. ------------- Marked as reviewed by matsaave (Committer). PR Review: https://git.openjdk.org/jdk/pull/17953#pullrequestreview-1925970472