RFR: 8310874: Runthese30m crashes with klass should be in the placeholders during verification [v2]

[Coleen Phillimore]

> For non-parallel capable class loaders, the class is not put in the placeholders table, so a verification at the wrong time will hit this assert. This change removes the invalid assert.
> While studying this code, we also noticed that if an error occurs during class loading after the klass is put in the LoaderConstraints table but before adding to the dictionary, we weren't removing the klass from the LoaderConstraints table.  This change also fixes this.
> Tested with inserting a LoaderConstraintTable::verify while running jck tests.  Also tested with failure insertion after the addClass() call to throw OutOfMemoryError.  Also tested with tier1-4.

Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:

  Add comments about remove_failed_loaded_class.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/15752/files
  - new: https://git.openjdk.org/jdk/pull/15752/files/bd32fa4c..5046e9e5

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=15752&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=15752&range=00-01

  Stats: 5 lines in 1 file changed: 4 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/15752.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15752/head:pull/15752

PR: https://git.openjdk.org/jdk/pull/15752