RFR: 8261479: CDS runtime code should check exceptions
    Calvin Cheung 
    ccheung at openjdk.java.net
       
    Fri Mar 19 21:20:52 UTC 2021
    
    
  
Proposed changes include:
- For the functions which could throw exceptions, the callers should pass in CHECK instead of THREAD.
- For the functions which won't throw exceptions, put THREAD as the first parameter.
      E.g.`static bool add_unregistered_class(Thread* current, InstanceKlass* k);`
- For the functions which won't throw exceptions and don't use THREAD, just simply get rid of the TRAPS parameter.
Testing: passed tiers 1 and 2, in-progress tiers 3 and 4.
-------------
Commit messages:
 - revert unnecessary changes to ArchiveUtils::log_to_classlist
 - merge
 - 8261479: CDS runtime code should check exceptions
Changes: https://git.openjdk.java.net/jdk/pull/3097/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3097&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8261479
  Stats: 65 lines in 13 files changed: 6 ins; 2 del; 57 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3097.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3097/head:pull/3097
PR: https://git.openjdk.java.net/jdk/pull/3097
    
    
More information about the hotspot-runtime-dev
mailing list