RFR: 8338693: assert(Atomic::add(&ik->_shared_class_load_count, 1) == 1) failed: shared class loaded more than once [v3]

[Matias Saavedra Silva]

> The test vmTestbase/nsk/sysdict/vm/stress/chain/chain007/chain007.java  occasionally fails due to the number of shared classes being incremented even though the shared class is not loaded. This patch moves the assert to only increment once loading is possible.

Matias Saavedra Silva has updated the pull request incrementally with two additional commits since the last revision:

 - Set loading failed in non-error conditions
 - Revert "Ioi suggestion"
   
   This reverts commit f60833d6bdd216c28cd7a3cbc697e8d4d318738a.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/20955/files
  - new: https://git.openjdk.org/jdk/pull/20955/files/f60833d6..56176ada

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

  Stats: 50 lines in 4 files changed: 22 ins; 22 del; 6 mod
  Patch: https://git.openjdk.org/jdk/pull/20955.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/20955/head:pull/20955

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