🐛 fix bug where output permutation was inferred incorrectly in heuristic mapper

burgholzer · burgholzer · commit 9a80fc490725 · 2023-01-13T08:19:39.000+01:00
Signed-off-by: Lukas Burgholzer &lt;lukas.burgholzer@jku.at&gt;
diff --git a/src/heuristic/HeuristicMapper.cpp b/src/heuristic/HeuristicMapper.cpp
@@ -176,8 +176,11 @@ void HeuristicMapper::map(const Configuration& configuration) {
             ++loc;
           }
           locations.at(target) = static_cast<std::int16_t>(loc);
+          qubits.at(loc)       = static_cast<std::int16_t>(target);
           op->setTargets({static_cast<qc::Qubit>(loc)});
-          qcMapped.initialLayout.at(target) = loc;
+          qcMapped.initialLayout.at(target)                       = loc;
+          qcMapped.outputPermutation[static_cast<qc::Qubit>(loc)] = target;
+          qcMapped.garbage.at(loc)                                = false;
         } else {
           op->setTargets({static_cast<qc::Qubit>(targetLocation)});
         }

Original file line number	Diff line number	Diff line change
`@@ -176,8 +176,11 @@ void HeuristicMapper::map(const Configuration& configuration) {`
`176`	`176`	`++loc;`
`177`	`177`	`}`
`178`	`178`	`locations.at(target) = static_cast<std::int16_t>(loc);`
	`179`	`+ qubits.at(loc) = static_cast<std::int16_t>(target);`
`179`	`180`	`op->setTargets({static_cast<qc::Qubit>(loc)});`
`180`		`- qcMapped.initialLayout.at(target) = loc;`
	`181`	`+ qcMapped.initialLayout.at(target) = loc;`
	`182`	`+ qcMapped.outputPermutation[static_cast<qc::Qubit>(loc)] = target;`
	`183`	`+ qcMapped.garbage.at(loc) = false;`
`181`	`184`	`} else {`
`182`	`185`	`op->setTargets({static_cast<qc::Qubit>(targetLocation)});`
`183`	`186`	`}`