JDK-8032656 (original) (raw)

Code Review for JDK-8032656

Prepared by: adlertz on Tue Jan 28 10:58:57 CET 2014
Workspace: /home/adlertz/repos/hotspot/JDK-8032656
Compare against: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/
Compare against version: 5828
Summary of changes: 144 lines changed: 124 ins; 0 del; 20 mod; 6148 unchg
Patch of changes: <JDK-8032656.patch>
Legend: Modified file Deleted file New file

[Cdiffs](src/share/vm/opto/chaitin.cpp.cdiff.html) [Udiffs](src/share/vm/opto/chaitin.cpp.udiff.html) [Sdiffs](src/share/vm/opto/chaitin.cpp.sdiff.html) [Frames](src/share/vm/opto/chaitin.cpp.frames.html) [Old](src/share/vm/opto/chaitin.cpp-.html) [New](src/share/vm/opto/chaitin.cpp.html) [Patch](src/share/vm/opto/chaitin.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/opto/chaitin.cpp) src/share/vm/opto/chaitin.cpp

rev 5829 : 8032656: Tag the MachSpillCopies with purpose information Summary: Subclassed the MachSpillCopyNode with different subnodes for different spill purposes to enhance debugging / visualization Reviewed-by:

1 line changed: 0 ins; 0 del; 1 mod; 2346 unchg

[Cdiffs](src/share/vm/opto/chaitin.hpp.cdiff.html) [Udiffs](src/share/vm/opto/chaitin.hpp.udiff.html) [Sdiffs](src/share/vm/opto/chaitin.hpp.sdiff.html) [Frames](src/share/vm/opto/chaitin.hpp.frames.html) [Old](src/share/vm/opto/chaitin.hpp-.html) [New](src/share/vm/opto/chaitin.hpp.html) [Patch](src/share/vm/opto/chaitin.hpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/opto/chaitin.hpp) src/share/vm/opto/chaitin.hpp

rev 5829 : 8032656: Tag the MachSpillCopies with purpose information Summary: Subclassed the MachSpillCopyNode with different subnodes for different spill purposes to enhance debugging / visualization Reviewed-by:

2 lines changed: 2 ins; 0 del; 0 mod; 692 unchg

[Cdiffs](src/share/vm/opto/coalesce.cpp.cdiff.html) [Udiffs](src/share/vm/opto/coalesce.cpp.udiff.html) [Sdiffs](src/share/vm/opto/coalesce.cpp.sdiff.html) [Frames](src/share/vm/opto/coalesce.cpp.frames.html) [Old](src/share/vm/opto/coalesce.cpp-.html) [New](src/share/vm/opto/coalesce.cpp.html) [Patch](src/share/vm/opto/coalesce.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/opto/coalesce.cpp) src/share/vm/opto/coalesce.cpp

rev 5829 : 8032656: Tag the MachSpillCopies with purpose information Summary: Subclassed the MachSpillCopyNode with different subnodes for different spill purposes to enhance debugging / visualization Reviewed-by:

3 lines changed: 0 ins; 0 del; 3 mod; 786 unchg

[Cdiffs](src/share/vm/opto/machnode.hpp.cdiff.html) [Udiffs](src/share/vm/opto/machnode.hpp.udiff.html) [Sdiffs](src/share/vm/opto/machnode.hpp.sdiff.html) [Frames](src/share/vm/opto/machnode.hpp.frames.html) [Old](src/share/vm/opto/machnode.hpp-.html) [New](src/share/vm/opto/machnode.hpp.html) [Patch](src/share/vm/opto/machnode.hpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/opto/machnode.hpp) src/share/vm/opto/machnode.hpp

rev 5829 : 8032656: Tag the MachSpillCopies with purpose information Summary: Subclassed the MachSpillCopyNode with different subnodes for different spill purposes to enhance debugging / visualization Reviewed-by:

120 lines changed: 120 ins; 0 del; 0 mod; 921 unchg

[Cdiffs](src/share/vm/opto/reg%5Fsplit.cpp.cdiff.html) [Udiffs](src/share/vm/opto/reg%5Fsplit.cpp.udiff.html) [Sdiffs](src/share/vm/opto/reg%5Fsplit.cpp.sdiff.html) [Frames](src/share/vm/opto/reg%5Fsplit.cpp.frames.html) [Old](src/share/vm/opto/reg%5Fsplit.cpp-.html) [New](src/share/vm/opto/reg%5Fsplit.cpp.html) [Patch](src/share/vm/opto/reg%5Fsplit.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/opto/reg%5Fsplit.cpp) src/share/vm/opto/reg_split.cpp

rev 5829 : 8032656: Tag the MachSpillCopies with purpose information Summary: Subclassed the MachSpillCopyNode with different subnodes for different spill purposes to enhance debugging / visualization Reviewed-by:

18 lines changed: 2 ins; 0 del; 16 mod; 1403 unchg


This code review page was prepared using /bin/webrev.ksh(vers 23.18-hg-never-dev).