[trivial] Add note on the freq
authorJihoon Lee <jhoon.it.lee@samsung.com>
Mon, 22 Nov 2021 13:40:57 +0000 (22:40 +0900)
committerJijoong Moon <jijoong.moon@samsung.com>
Wed, 1 Dec 2021 09:53:22 +0000 (18:53 +0900)
This patch add note on the frequency map

**Self evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee <jhoon.it.lee@samsung.com>
nntrainer/compiler/multiout_realizer.cpp

index ea639a8..fd9d98e 100644 (file)
@@ -56,6 +56,12 @@ MultioutRealizer::realize(const GraphRepresentation &reference) {
   for (auto &[con_name, freq] : freq_map) {
     props::InputConnection con;
     from_string(con_name, con);
+
+    /// @note freq < 1 should never happen as the map entry is not created.
+    /// but if it happens multiout realizer is not interested in checking if it
+    /// is a dangled or actually an output. So there is no assurance done at
+    /// this point. Some other class must check if the given graph is formed in
+    /// a correct way.
     if (freq <= 1) {
       continue;
     }