account for empty memory size

llvm · Nov 17, 2024 · a104870 · a104870
1 parent d803cb0
commit a104870
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/lib/Conversion/SCFToCalyx/SCFToCalyx.cpp b/lib/Conversion/SCFToCalyx/SCFToCalyx.cpp
@@ -322,8 +322,10 @@ class BuildOpGroups : public calyx::FuncOpPartialLoweringPattern {
           jsonOS.value(getState<ComponentLoweringState>().getExtMemData());
           jsonOS.flush();
           outFile.close();
-        } else
+        } else {
           llvm::errs() << "Unable to open file for writing\n";
+          return failure();
+        }
       }
     }
 
@@ -733,6 +735,11 @@ LogicalResult BuildOpGroups::buildOp(PatternRewriter &rewriter,
 void insertNestedArrayValue(llvm::json::Array &array,
                             const std::vector<int> &indices, size_t index,
                             llvm::json::Value value) {
+  if (indices.empty()) {
+    array.emplace_back(std::move(value));
+    return;
+  }
+
   if (index == indices.size() - 1) {
     // ensure the array is big enough
     while (array.size() <= static_cast<size_t>(indices[index])) {