United StatesChange Country, Oracle Worldwide Web Sites Communities I am a... I want to...
Bug ID: JDK-6470497 early C2 cleanups for Java 7
JDK-6470497 : early C2 cleanups for Java 7

Details
Type:
Bug
Submit Date:
2006-09-14
Status:
Resolved
Updated Date:
2010-04-02
Project Name:
JDK
Resolved Date:
2006-11-14
Component:
hotspot
OS:
generic
Sub-Component:
compiler
CPU:
generic
Priority:
P4
Resolution:
Fixed
Affected Versions:
7
Fixed Versions:
hs10 (b03)

Related Reports
Backport:
Backport:
Relates:
Relates:
Relates:
Relates:
Relates:
Relates:

Sub Tasks

Description
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.

                                    

Comments
EVALUATION

Small fixes.
                                     
2006-11-14
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.
                                     
2006-12-10



Hardware and Software, Engineered to Work Together