♻️ 🔥 use opToSingleGateMatrix to retrieve matrix and remove unused CX…

…_MAT and CZ_MAT
cda-tum · Dec 7, 2024 · 4632650 · 4632650
1 parent 3257484
commit 4632650
Show file tree

Hide file tree

Showing 4 changed files with 261 additions and 279 deletions.
diff --git a/src/dd/NoiseFunctionality.cpp b/src/dd/NoiseFunctionality.cpp
@@ -180,16 +180,11 @@ mEdge StochasticNoiseFunctionality::generateNoiseOperation(
           stackOperation(operation, target, effect, amplitudeDampingMatrix);
       break;
     }
-    case (qc::X): {
-      operation = stackOperation(operation, target, effect, X_MAT);
-      break;
-    }
-    case (qc::Y): {
-      operation = stackOperation(operation, target, effect, Y_MAT);
-      break;
-    }
+    case (qc::X):
+    case (qc::Y):
     case (qc::Z): {
-      operation = stackOperation(operation, target, effect, Z_MAT);
+      operation = stackOperation(operation, target, effect,
+                                 opToSingleGateMatrix(effect));
       break;
     }
     default: {

diff --git a/src/dd/Simulation.cpp b/src/dd/Simulation.cpp
@@ -293,7 +293,8 @@ void extractProbabilityVectorRecursive(const QuantumComputation* qc,
 
       if (RealNumber::approximatelyEquals(pone, 1.)) {
         const qc::MatrixDD xGate =
-            dd.makeGateDD(X_MAT, static_cast<Qubit>(permutation.at(target)));
+            dd.makeGateDD(opToSingleGateMatrix(qc::X),
+                          static_cast<Qubit>(permutation.at(target)));
         state = dd.applyOperation(xGate, state);
         continue;
       }

diff --git a/test/dd/test_edge_functionality.cpp b/test/dd/test_edge_functionality.cpp
@@ -12,6 +12,7 @@
 #include "dd/Node.hpp"
 #include "dd/Package.hpp"
 #include "dd/RealNumber.hpp"
+#include "ir/operations/OpType.hpp"
 
 #include <cmath>
 #include <cstddef>
@@ -373,7 +374,7 @@ TEST(DensityMatrixFunctionality, GetValueByIndexProperDensityMatrix) {
   const auto nqubits = 1U;
   auto dd = std::make_unique<dd::Package<>>(nqubits);
   auto zero = dd->makeZeroDensityOperator(nqubits);
-  const auto op1 = dd->makeGateDD(dd::H_MAT, 0U);
+  const auto op1 = dd->makeGateDD(dd::opToSingleGateMatrix(qc::H), 0U);
   const auto op2 = dd->makeGateDD(dd::rzMat(dd::PI_4), 0U);
   auto state = dd->applyOperationToDensity(zero, op1);
   state = dd->applyOperationToDensity(state, op2);
@@ -407,7 +408,7 @@ TEST(DensityMatrixFunctionality, GetSparseMatrixConsistency) {
   const auto nqubits = 1U;
   auto dd = std::make_unique<dd::Package<>>(nqubits);
   auto zero = dd->makeZeroDensityOperator(nqubits);
-  const auto op1 = dd->makeGateDD(dd::H_MAT, 0U);
+  const auto op1 = dd->makeGateDD(dd::opToSingleGateMatrix(qc::H), 0U);
   const auto op2 = dd->makeGateDD(dd::rzMat(dd::PI_4), 0U);
   auto state = dd->applyOperationToDensity(zero, op1);
   state = dd->applyOperationToDensity(state, op2);
@@ -437,7 +438,7 @@ TEST(DensityMatrixFunctionality, PrintMatrix) {
   const auto nqubits = 1U;
   auto dd = std::make_unique<dd::Package<>>(nqubits);
   auto zero = dd->makeZeroDensityOperator(nqubits);
-  const auto op1 = dd->makeGateDD(dd::H_MAT, 0U);
+  const auto op1 = dd->makeGateDD(dd::opToSingleGateMatrix(qc::H), 0U);
   const auto op2 = dd->makeGateDD(dd::rzMat(dd::PI_4), 0U);
   auto state = dd->applyOperationToDensity(zero, op1);
   state = dd->applyOperationToDensity(state, op2);