RFR: 8261479: CDS runtime code should check exceptions [v4]

[Calvin Cheung]

> 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.

Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:

  @dholmes-ora review comments

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/3097/files
  - new: https://git.openjdk.java.net/jdk/pull/3097/files/815b6c4a..417d10c1

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3097&range=03
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3097&range=02-03

  Stats: 8 lines in 2 files changed: 0 ins; 2 del; 6 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