RFR: 8253923: C2 doesn't always run loop opts for compilations that include loops

Roland Westrelin roland at openjdk.java.net
Thu Oct 8 09:43:47 UTC 2020


On Thu, 8 Oct 2020 09:37:34 GMT, Nils Eliasson <neliasso at openjdk.org> wrote:

> > I updated the change in place (force pushed).
> > https://openjdk.github.io/cr/?repo=jdk&pr=478&range=00 is the new change.
> 
> That looks exactly like the code I reviewed.

The initial change would set the has_loops flag before inlining. The current one sets it when a new method is parsed
(so once inlining has happened). In any case, I take it you're fine with the current change. Thanks for the review.

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

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


More information about the hotspot-compiler-dev mailing list