RFR: 8320128: Clean up Parse constructor for OSR [v3]

[Xin Liu]

> There's a special case for the constructor of Parse. If current compilation is OSR and it is handling the top-level method(depth() == 1), then
> 
> 1. _tf = C->tf();
> 2. _entry_bci = C->entry_bci();
> 3. _flow = method()->get_osr_flow_analysis(_entry_bci);
> 
> We don't need to assign those member data twice. We can also factor out _flow->failing() for the special case and normal cases. 
> 
> It's worth mentioning that we can't save ciTypeFlow computation because
> get_osr_flow_analysis(_entry_bci) actually needs get_flow_analysis(method()).

Xin Liu has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:

 - Use atomic logline and resume #ifdef ASSERT.
 - Merge branch 'master' into JDK-8320128
 - Update according to reviewer's feedback.
 - 8320128: Clean up Parse constructor for OSR

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/16669/files
  - new: https://git.openjdk.org/jdk/pull/16669/files/1f7c956c..ec89638c

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

  Stats: 789833 lines in 4137 files changed: 177560 ins; 537711 del; 74562 mod
  Patch: https://git.openjdk.org/jdk/pull/16669.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16669/head:pull/16669

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