Code Review for 6758234

Prepared by: Thomas Rodriguez (never) on Fri Nov 7 13:16:51 PST 2008
Workspace:/net/smite.sfbay/export/ws/c1
Compare against: ssh://hg.openjdk.java.net/jdk7/hotspot-comp-gate/hotspot
Compare against version:411
Summary of changes: 42 lines changed: 40 ins; 2 del; 0 mod; 1068 unchg
Patch of changes: 6758234.patch
Author comments:
6758234: if (k cond (a ? : b: c)) returns reversed answer if k is constant and b and c are longs
Reviewed-by:

When folding up tests against the result of the ternary operator we
would double swap the result in the constant was on the left instead
of on the right as we expected. The fix is simply only swap once.

Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/c1/c1_Optimizer.cpp

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

------ ------ ------ ------ --- New Patch Raw test/compiler/6758234/Test6758234.java

40 lines changed: 40 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /net/smite.sfbay/never/bin/hgwebrev (vers 23.12-hg-never).