RFR: 8227745, 8233915: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents

[Lindenmaier, Goetz]

Hi Richard, 

I think you can add a second bug fixed by your pull request by 
issueing comment 
  /issue JDK-8233915
in the pull request.

Erik, please correct me if wrong.
(The change fixes/implements two JBS issues.)

Best regards,
  Goetz.

> -----Original Message-----
> From: hotspot-dev <hotspot-dev-retn at openjdk.java.net> On Behalf Of Erik
> Helin
> Sent: Friday, September 11, 2020 11:58 AM
> To: hotspot-dev at openjdk.java.net
> Subject: Re: RFR: 8227745, 8233915: Enable Escape Analysis for Better
> Performance in the Presence of JVMTI Agents
> 
> On Fri, 11 Sep 2020 09:54:06 GMT, Erik Helin <ehelin at openjdk.org> wrote:
> 
> >>>
> >>>
> >>> @reinrich This change now passes all automated pre-integration checks.
> In addition to the automated checks, the change
> >>> must also fulfill all [project specific
> requirements](https://github.com/openjdk/jdk/blob/master/CONTRIBUTING
> .md)
> >>> After integration, the commit message will be:
> >>>
> >>> ```
> >>> 8227745: 8233915: Enable Escape Analysis for Better Performance in the
> Presence of JVMTI Agents
> >>> 8233915: JVMTI FollowReferences: Java Heap Leak not found because
> of C2 Scalar Replacement
> >>>
> >>> Reviewed-by: mdoerr
> >>> ```
> >>
> >> The commit message looks odd to me. Should I remove ", 8233915" from
> the title of this request? I used the old scheme
> >> "JBS-Id[, JBS-Id]*" and this might be wrong.
> >
> > The commit message is correct according to [JEP
> 357](http://openjdk.java.net/jeps/357)
> 
> Sorry, now I see. Yes, please remove `, 8233915` from the title!
> 
> -------------
> 
> PR: https://git.openjdk.java.net/jdk/pull/119