[onert] Use getUses() copy instead of reference

This commit fixes the problem that occurs in loop traversal due to change of list by using copy instead of reference. ONE-DCO-1.0-Signed-off-by: Jiyoung Yun <[email protected]>
Samsung · Sep 4, 2024 · 4469012 · 4469012
1 parent c66e299
commit 4469012
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/runtime/onert/core/src/compiler/train/pass/TrainableConstantInsertionPass.cc b/runtime/onert/core/src/compiler/train/pass/TrainableConstantInsertionPass.cc
@@ -40,7 +40,7 @@ void TrainableConstantInsertionPass::callback(const ir::OperationIndex &node_ind
       continue;
 
     // Insert new operands for shared constant except for the current node.
-    const auto &uses = object.getUses();
+    const auto uses(object.getUses());
     for (const auto &use_index : uses)
     {
       if (use_index == node_index)

diff --git a/runtime/onert/core/src/ir/train/UseDefGenerator.cc b/runtime/onert/core/src/ir/train/UseDefGenerator.cc
@@ -425,7 +425,7 @@ void UseDefGenerator::initForForwardingNodes()
     }
 
     assert(_training_usedefs.at(forwarding_operand_index).getTrainingUses().size() == 0);
-    const auto &uses = operand.getUses();
+    const auto uses(operand.getUses());
     for (const auto &use : uses)
       insertUse(forwarding_operand_index, TrainingOperationIndex{use, is_forward});
   });