RFR: 8282024: add EscapeAnalysis statistics under PrintOptoStatistics [v7]

[aamarsh]

On Tue, 3 May 2022 18:50:50 GMT, Xin Liu <xliu at openjdk.org> wrote:

>> aamarsh has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
>> 
>>   adding escape analysis and scalar replacement statistics
>
> src/hotspot/share/opto/compile.cpp line 2215:
> 
>> 2213: #ifndef PRODUCT
>> 2214:     Atomic::add(&ConnectionGraph::_no_escape_counter, total_scalar_replaced);
>> 2215:     Atomic::add(&ConnectionGraph::_no_escape_counter, _local_no_escape_ctr);
> 
> you can merge two atomic operations into one.  Atomic::add(&ConnectionGraph::_no_escape_counter(&ConnectionGraph::_no_escape_counter, _local_no_escape_ctr + total_scalar_replaced); 
> 
> This is smart. last revision is the final revision.

@navyxliu Thank you so much for your feedback, it is helpful for me :).

As per this issue, since no nodes are ever removed from the list, I think this issue is avoided. If we find a loop back to a node we have already searched and attempt to push this node at line 2620, it will not be added to the list because it already exists.

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

PR: https://git.openjdk.java.net/jdk/pull/8019