You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@systemml.apache.org by mb...@apache.org on 2016/02/03 22:59:44 UTC
[2/2] incubator-systemml git commit: Fix simplification rewrite
'remove unnecessary reorg' (op equivalence)
Fix simplification rewrite 'remove unnecessary reorg' (op equivalence)
Project: http://git-wip-us.apache.org/repos/asf/incubator-systemml/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-systemml/commit/6d05ab6d
Tree: http://git-wip-us.apache.org/repos/asf/incubator-systemml/tree/6d05ab6d
Diff: http://git-wip-us.apache.org/repos/asf/incubator-systemml/diff/6d05ab6d
Branch: refs/heads/master
Commit: 6d05ab6d6bdc8d7d78f4608c18f0bf9bc9205fd5
Parents: 428fcda
Author: Matthias Boehm <mb...@us.ibm.com>
Authored: Tue Feb 2 17:15:22 2016 -0800
Committer: Matthias Boehm <mb...@us.ibm.com>
Committed: Wed Feb 3 13:59:23 2016 -0800
----------------------------------------------------------------------
.../sysml/hops/rewrite/RewriteAlgebraicSimplificationStatic.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-systemml/blob/6d05ab6d/src/main/java/org/apache/sysml/hops/rewrite/RewriteAlgebraicSimplificationStatic.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/sysml/hops/rewrite/RewriteAlgebraicSimplificationStatic.java b/src/main/java/org/apache/sysml/hops/rewrite/RewriteAlgebraicSimplificationStatic.java
index 2368153..77b2e5e 100644
--- a/src/main/java/org/apache/sysml/hops/rewrite/RewriteAlgebraicSimplificationStatic.java
+++ b/src/main/java/org/apache/sysml/hops/rewrite/RewriteAlgebraicSimplificationStatic.java
@@ -1378,8 +1378,9 @@ public class RewriteAlgebraicSimplificationStatic extends HopRewriteRule
if( hi instanceof ReorgOp && HopRewriteUtils.isValidOp(((ReorgOp)hi).getOp(), lookup) ) //first reorg
{
+ ReOrgOp firstOp = ((ReorgOp)hi).getOp();
Hop hi2 = hi.getInput().get(0);
- if( hi2 instanceof ReorgOp && HopRewriteUtils.isValidOp(((ReorgOp)hi2).getOp(), lookup) ) //second reorg
+ if( hi2 instanceof ReorgOp && ((ReorgOp)hi2).getOp()==firstOp ) //second reorg w/ same type
{
Hop hi3 = hi2.getInput().get(0);
//remove unnecessary chain of t(t())