add comments

Signed-off-by: Chen Tong <chentong@us.ibm.com>
onnx · chentong319 · Jan 29, 2025 · Jan 15, 2025 · Jan 15, 2025 · Jan 15, 2025
commit 069467887b2b548819addb048b8b2fa01989d649
diff --git a/MLIR.cmake b/MLIR.cmake
@@ -1,6 +1,9 @@
 # SPDX-License-Identifier: Apache-2.0
 
 if (ONNX_MLIR_ENABLE_PYRUNTIME_LIT)
+  # This function is defined in llvm_project.
+  # Define a dummy function for PYRUNTIME_LIT.
+  # If needed, the definition from llvm_project can be copied.
   function(llvm_update_compile_flags name)
   endfunction()
 else()

diff --git a/src/Runtime/ExecutionSession.hpp b/src/Runtime/ExecutionSession.hpp
@@ -21,6 +21,11 @@
 #include <string>
 
 #include "OnnxMlirRuntime.h"
+
+// LLVM provides the wrapper class, llvm::sys::DynamicLibrary, for dynamic
+// library. When PYRUNTIME_LIT is built without the LLVM, the handle type for
+// dynamic library in Linux is used. DynamicLibraryHandleType is defined for
+// the two cases.
 #ifndef ENABLE_PYRUNTIME_LIT
 #include "llvm/Support/DynamicLibrary.h"
 typedef llvm::sys::DynamicLibrary DynamicLibraryHandleType;