From 047d66a657cb71f762ef49c01e3c3f05b253168b Mon Sep 17 00:00:00 2001
From: Wilson Snyder <wsnyder@wsnyder.org>
Date: Tue, 25 Sep 2012 19:17:09 -0400
Subject: [PATCH] Slice code cleanup.  No functional change.

---
 src/V3Slice.cpp | 28 ++++++----------------------
 1 file changed, 6 insertions(+), 22 deletions(-)

diff --git a/src/V3Slice.cpp b/src/V3Slice.cpp
index 1980e5866..9539bc09f 100644
--- a/src/V3Slice.cpp
+++ b/src/V3Slice.cpp
@@ -54,14 +54,6 @@ class SliceCloneVisitor : public AstNVisitor {
     //  AstNodeAssign::user2()	    -> int. The number of clones needed for this assign
     //  AstArraySel::user3()	    -> bool.  Error detected
 
-    // ENUMS
-    enum RedOp {		// The type of unary operation to be expanded
-	REDOP_UNKNOWN,		// Unknown/Unsupported
-	REDOP_OR,		// Or Reduction
-	REDOP_AND,		// And Reduction
-	REDOP_XOR,		// Xor Reduction
-	REDOP_XNOR};		// Xnor Reduction
-
     // STATE
     vector<vector<unsigned> >	m_selBits;	// Indexes of the ArraySel we are expanding
     int				m_vecIdx;	// Current vector index
@@ -140,14 +132,6 @@ class SliceCloneVisitor : public AstNVisitor {
 	m_selBits.clear();
 	UINFO(4, "Cloning "<<nodep->user2()<<" times: "<<nodep<<endl);
 
-	// Figure out what type of operation this is so we don't have to cast on
-	// every clone.
-	RedOp redOpType = REDOP_UNKNOWN;
-	if (nodep->castRedOr()) redOpType = REDOP_OR;
-	else if (nodep->castRedAnd()) redOpType = REDOP_AND;
-	else if (nodep->castRedXor()) redOpType = REDOP_XOR;
-	else if (nodep->castRedXnor()) redOpType = REDOP_XNOR;
-
 	AstNode* lhsp = NULL;
 	AstNode* rhsp = NULL;
 	for (int i = 0; i < nodep->user2(); ++i) {
@@ -158,20 +142,20 @@ class SliceCloneVisitor : public AstNVisitor {
 	    if (!lhsp) lhsp = clonep;
 	    else rhsp = clonep;
 	    if (lhsp && rhsp) {
-		switch (redOpType) {
-		case REDOP_OR:
+		switch (nodep->type()) {
+		case AstType::atREDOR:
 		    lhsp = new AstOr(nodep->fileline(), lhsp, rhsp);
 		    break;
-		case REDOP_AND:
+		case AstType::atREDAND:
 		    lhsp = new AstAnd(nodep->fileline(), lhsp, rhsp);
 		    break;
-		case REDOP_XOR:
+		case AstType::atREDXOR:
 		    lhsp = new AstXor(nodep->fileline(), lhsp, rhsp);
 		    break;
-		case REDOP_XNOR:
+		case AstType::atREDXNOR:
 		    lhsp = new AstXnor(nodep->fileline(), lhsp, rhsp);
 		    break;
-		default: // REDOP_UNKNOWN
+		default:
 		    nodep->v3fatalSrc("Unsupported: Unary operation on multiple packed dimensions");
 		    break;
 		}