RFR: 8264711: More runtime TRAPS cleanups [v4]

Harold Seigel hseigel at openjdk.java.net
Wed Apr 7 14:24:24 UTC 2021


On Mon, 5 Apr 2021 23:15:48 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Harold Seigel has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   remove unneeded statement
>
> src/hotspot/share/classfile/klassFactory.cpp line 117:
> 
>> 115:                                                    ClassLoaderData* loader_data,
>> 116:                                                    Handle protection_domain,
>> 117:                                                    JvmtiCachedClassFileData** cached_class_file) {
> 
> I don't think this removal of TRAPS is correct. The  ClassFileLoadHook could result in an exception becoming pending.

Thanks!  This change was reverted.

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

PR: https://git.openjdk.java.net/jdk/pull/3345


More information about the hotspot-dev mailing list