summaryrefslogtreecommitdiff
path: root/nn/runtime/ExecutionPlan.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'nn/runtime/ExecutionPlan.cpp')
-rw-r--r--nn/runtime/ExecutionPlan.cpp5
1 files changed, 5 insertions, 0 deletions
diff --git a/nn/runtime/ExecutionPlan.cpp b/nn/runtime/ExecutionPlan.cpp
index 1f540805f..1797475ac 100644
--- a/nn/runtime/ExecutionPlan.cpp
+++ b/nn/runtime/ExecutionPlan.cpp
@@ -555,6 +555,11 @@ int ExecutionStep::finishStepModel(const ModelBuilder* mainModel, bool* hasOutpu
[](auto& e) { return e.second; });
NN_RETURN_IF_ERROR(mStepModel.identifyInputsAndOutputs(inputs.size(), inputs.data(),
outputs.size(), outputs.data()));
+ // TODO: Model::finish() should use ValidationMode::RUNTIME when sending the
+ // step model to CpuDevice. Right now, this is harmless because the only
+ // difference in validation occurs with control flow operations and inputs
+ // or outputs of unknown size and we never send control flow operations to
+ // CpuDevice. We need to address this if this behavior changes (b/151634976).
NN_RETURN_IF_ERROR(mStepModel.finish());
// TODO: Move compilation elsewhere?