JDK-8223137 : Rename predicate 'do_unroll_only()' to 'is_unroll_only()'.
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: 11,13
  • Priority: P5
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2019-04-30
  • Updated: 2022-02-06
  • Resolved: 2019-05-03
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 11 JDK 13
11.0.14-oracleFixed 13 b20Fixed
Related Reports
Relates :  
JDK-8216137 - assert(Compile::current()->live_nodes() < Compile::current()->max_node_limit()) failed: Live Node limit exceeded limit
Description
The "do" prefix is used as a command ditto elsewhere, while predicates typically use "is".
Comments
Fix request [11u] I downport this for parity with 11.0.14-oracle. Clean backport. Only renames a function, very low risk. Nighttests pass.
30-09-2021
Git URL: https://github.com/openjdk/jdk/commit/fcbca8224766a41659774acff0145dc1aee7d52f
29-09-2021
URL: http://hg.openjdk.java.net/jdk/jdk/rev/3ebf58dbf5d8 User: phedlin Date: 2019-05-03 09:00:17 +0000
03-05-2019
Patric, I would prefer merging all these cleanup RFEs (JDK-8223137 - JDK-8223143) into one to lower the reviewing / bug management overhead. You could call it "Loop optimization code cleanup" or similar.
30-04-2019
Okay, fine with me.
30-04-2019
This was actually my initial intent (or even to just have them committed as prerequisites) but after some consideration I thought it might be more straight forward with a set of (mostly) trivial changes (single review/er items). They are all optional if someone should object and there is a WS clean-up that is rather disruptive, even though it's just a single file. Still, if you feel strongly in favour of a single RFE I can certainly go that way.
30-04-2019