JDK-6470497 : early C2 cleanups for Java 7
  • Type: Bug
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: 7
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • OS: generic
  • CPU: generic
  • Submitted: 2006-09-14
  • Updated: 2010-04-02
  • Resolved: 2006-11-14
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 6 JDK 7 Other
6u4Fixed 7Fixed hs10Fixed
Related Reports
Relates :  
Relates :  
Relates :  
Relates :  
Relates :  
Relates :  
There are a number of low-risk cleanups and refactorings needed in the C2 code which need to precede work on 6337834, 6428387, and 6461827.  They disturb the source base and obscure the algorithmic changes necessary to fix those bugs.  I'm putting them back separately to ease maintenance and analysis.

SUGGESTED FIX These cleanups were committed in six putbacks. The webrevs, with associated comments, are as follows. Webrev: http://analemma.sfbay.sun.com/net/prt-archiver.sfbay/data/archived_workspaces/main/c2_baseline/2006/20060929154458.jrose.dolphin-cleanups/workspace/webrevs/webrev-2006.09.29/index.html Partial 6470497: early C2 cleanups for Java 7 (part A of four) Machine specific cleanups, including CastP2X nodes. Webrev: http://analemma.sfbay.sun.com/net/prt-archiver.sfbay/data/archived_workspaces/main/c2_baseline/2006/20061002120630.jrose.dolphin-cleanups/workspace/webrevs/webrev-2006.10.02/index.html Partial 6470497: early C2 cleanups for Java 7 (part B of four or more) Cleanups to the CI. Also, add nonstatic field lists to instance klasses. Webrev: http://analemma.sfbay.sun.com/net/prt-archiver.sfbay/data/archived_workspaces/main/c2_baseline/2006/20061003112536.jrose.dolphin-cleanups/workspace/webrevs/webrev-2006.10.03/index.html Partial 6470497: early C2 cleanups for Java 7 (part C of four or more) Cleanups to debugging functionality. Includes some improvements to 6175499 (nmethod profiling information). Archived data: /net/prt-archiver.sfbay/data/archived_workspaces/main/c2_baseline/2006/20061012181105.jrose.dolphin-cleanups/ Webrev: http://analemma.sfbay.sun.com/net/prt-archiver.sfbay/data/archived_workspaces/main/c2_baseline/2006/20061012181105.jrose.dolphin-cleanups/workspace/webrevs/webrev-2006.10.12/index.html Partial 6470497: early C2 cleanups for Java 7 (part D of seven or more) Minor enhancements and refactorings to calls, allocation sites, and memory barriers. Record adr_type more consistently everywhere. Webrev: http://analemma.sfbay.sun.com/net/prt-archiver.sfbay/data/archived_workspaces/main/c2_baseline/2006/20061023120554.jrose.dolphin-cleanups/workspace/webrevs/webrev-2006.10.23/index.html Partial 6470497: early C2 cleanups for Java 7 (part E of about seven) Refactorings. Should not affect compiler output, with very minor exceptions: - Slightly shift some uncommon trap conditions as noted in parse2.cpp. - The parser's map is disconnected more eagerly (GK::stop_and_kill_map). - Remove a loophole in a pattern match (is_range_check) in ifnode.cpp. - Remove a loophole in the handling of CMove in loopopts.cpp. Webrev: http://analemma.sfbay.sun.com/net/prt-archiver.sfbay/data/archived_workspaces/main/c2_baseline/2006/20061118180600.jrose.dolphin-cleanups/workspace/webrevs/webrev-2006.11.18/index.html Fixed 6470497: early C2 cleanups for Java 7 (part G of six). This is the final part of six: A,B,C,D,E,G. A seventh part (F) has been reserved for a separate bug.

EVALUATION Small fixes.