Code Review for 6795362

Prepared by: Christian Thalinger (twisti) on Mon Feb 2 20:39:02 CET 2009
Workspace:/export/home/twisti/hotspot-comp/6795362
Compare against: /export/home/twisti/hotspot-comp/hotspot
Compare against version:515
Summary of changes: 66 lines changed: 51 ins; 0 del; 15 mod; 2445 unchg
Patch of changes: 6795362.patch
Author comments:
6795362: 32bit server compiler leads to wrong results on solaris-x86
Summary: The C2 compiler leads to wrong results on solaris-i486 (32-bit) for a testcase given in the CR.
Reviewed-by: ??

I also fixed the same bug in AndINode::Identity but I was unable to write a testcase for it. To be more consistent, I renamed the shift_con value to shift.
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/opto/mulnode.cpp

rev 516 : 6795362: 32bit server compiler leads to wrong results on solaris-x86
Summary: The C2 compiler leads to wrong results on solaris-i486 (32-bit) for a testcase given in the CR.
Reviewed-by: ??
16 lines changed: 2 ins; 0 del; 14 mod; 1323 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/utilities/globalDefinitions.hpp

rev 516 : 6795362: 32bit server compiler leads to wrong results on solaris-x86
Summary: The C2 compiler leads to wrong results on solaris-i486 (32-bit) for a testcase given in the CR.
Reviewed-by: ??
2 lines changed: 1 ins; 0 del; 1 mod; 1122 unchg

------ ------ ------ ------ --- New Patch Raw test/compiler/6795362/Test6795362.java

rev 516 : 6795362: 32bit server compiler leads to wrong results on solaris-x86
Summary: The C2 compiler leads to wrong results on solaris-i486 (32-bit) for a testcase given in the CR.
Reviewed-by: ??
48 lines changed: 48 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /export/home/twisti/bin/webrev (vers 23.10-hg).