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 :  
JDK-6539438 - raise_LCA_above_marks stops too early
Relates :  
JDK-6544710 - Performance regression in RangeCheckMicroBenchmark
Relates :  
JDK-6565458 - 10x performance regression on x86 in micro test with Reflection.
Relates :  
JDK-6832293 - JIT compiler got wrong result in type checking with -server
Relates :  
JDK-6843752 - missing code for an anti-dependent Phi in GCM
Relates :  
JDK-6592707 - REGRESSION: Java 64-Bit Server VM sparcv9 crash with unhandled implicit exception in compiled code
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
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.
10-12-2006
EVALUATION Small fixes.
14-11-2006