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