Update latest FOEDAG and return status of CFG registration (#466)

Follow up from #1200. Now the FOEDAG code will not do assertion and will return status. Again, whatever status is returned (true or false), the main function will just ignore it. Update the FOEDAG to latest
os-fpga · Aug 1, 2023 · 3c7ddc2 · 3c7ddc2
2 parents ea87724 + 7bd6253
commit 3c7ddc2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/FOEDAG b/FOEDAG
diff --git a/src/Compiler/CompilerRS.cpp b/src/Compiler/CompilerRS.cpp
@@ -628,8 +628,9 @@ bool CompilerRS::RegisterCommands(TclInterpreter *interp, bool batchMode) {
     };
     interp->registerCmd("assembler", assembler, this, 0);
   }
-  cfgcompiler->RegisterCallbackFunction("assembler", BitAssembler_entry);
-  return true;
+  bool status =
+      cfgcompiler->RegisterCallbackFunction("assembler", BitAssembler_entry);
+  return status;
 }
 
 std::string CompilerRS::BaseVprCommand() {
+1 −1		CMakeLists.txt
+18 −2		src/Configuration/CFGCommon/CFGCommon.cpp
+2 −0		src/Configuration/CFGCommon/CFGCommon.h
+21 −4		src/Configuration/CFGCompiler/CFGCompiler.cpp
+29 −1		tests/unittest/CFGCommon/CFGCommon_test.cpp
+21 −0		tests/unittest/CFGCompiler/CFGCompiler_test.cpp