Code Review for graal

Prepared by:vvenkata on Thu Dec 12 12:32:51 CST 2013
Workspace:/home/tester/graalrepos/intrinsics-exp/graal
Compare against: http://10.181.37.176:8080/scm/hg/graal/graal
Compare against version:default
Branch:intrinsics-exp
Summary of changes: 1225 lines changed: 1161 ins; 61 del; 3 mod; 3191 unchg
Patch of changes: graal.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw graal/com.oracle.graal.compiler.hsail.test.infra/src/com/oracle/graal/compiler/hsail/test/infra/KernelTester.java

rev 13944 : Change to intrinsify Math.abs(double x)

Added test cases for Math.abs(double x) and math.sqrt(double x)

Also fixed a bug in Kerneltester which was reporting test case failure when the expected and actual results were both NaN.
4 lines changed: 4 ins; 0 del; 0 mod; 913 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw graal/com.oracle.graal.compiler.hsail/src/com/oracle/graal/compiler/hsail/HSAILLIRGenerator.java

rev 13953 : Changes to remove the enum in HSAILMathIntrinsicsNode.
52 lines changed: 51 ins; 0 del; 1 mod; 786 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw graal/com.oracle.graal.hotspot.hsail/src/com/oracle/graal/hotspot/hsail/HSAILHotSpotBackend.java

rev 14070 : Merge with default
17 lines changed: 17 ins; 0 del; 0 mod; 304 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw graal/com.oracle.graal.hotspot.hsail/src/com/oracle/graal/hotspot/hsail/HSAILHotSpotReplacementsImpl.java

rev 13953 : Changes to remove the enum in HSAILMathIntrinsicsNode.
0 lines changed: 0 ins; 0 del; 0 mod; 80 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw graal/com.oracle.graal.lir.hsail/src/com/oracle/graal/lir/hsail/HSAILArithmetic.java

rev 14070 : Merge with default
30 lines changed: 27 ins; 2 del; 1 mod; 423 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw mx/projects

rev 14070 : Merge with default
9 lines changed: 8 ins; 0 del; 1 mod; 685 unchg

------ ------ ------ ------ ------ --- New Patch Raw graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/DoubleAbsTest.java

rev 13951 : Added comments. Expanded some of the test cases with additional corner cases.
92 lines changed: 92 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/DoubleCeilTest.java

rev 13951 : Added comments. Expanded some of the test cases with additional corner cases.
94 lines changed: 94 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/DoubleFloorTest.java

rev 13954 : Some code cleanup and fixed a bug in one of hte test cases.
94 lines changed: 94 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/DoubleRintTest.java

rev 13951 : Added comments. Expanded some of the test cases with additional corner cases.
95 lines changed: 95 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/DoubleSqrtTest.java

rev 13951 : Added comments. Expanded some of the test cases with additional corner cases.
94 lines changed: 94 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/FloatAbsTest.java

rev 13951 : Added comments. Expanded some of the test cases with additional corner cases.
92 lines changed: 92 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/IntAbsTest.java

rev 13951 : Added comments. Expanded some of the test cases with additional corner cases.
86 lines changed: 86 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/LongAbsTest.java

rev 13951 : Added comments. Expanded some of the test cases with additional corner cases.
86 lines changed: 86 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw graal/com.oracle.graal.replacements.hsail/src/com/oracle/graal/replacements/hsail/HSAILMathIntrinsicsNode.java

rev 13954 : Some code cleanup and fixed a bug in one of hte test cases.
194 lines changed: 194 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New Patch Raw graal/com.oracle.graal.replacements.hsail/src/com/oracle/graal/replacements/hsail/HSAILMathSubstitutions.java

rev 13953 : Changes to remove the enum in HSAILMathIntrinsicsNode.
127 lines changed: 127 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ Old --- Patch --- graal/com.oracle.graal.compiler.hsail.test/src/com/oracle/graal/compiler/hsail/test/FloatSqrtTest.java

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

This code review page was prepared using /bin/webrev.ksh (vers 24.0-hg+jbs).