Fix static gc-opt build & add comments for components registration

Author: kurapov-peter

src/gc-opt/CMakeLists.txt

@@ -26,12 +26,15 @@ if(GC_DEV_LINK_LLVM_DYLIB)
   )
   get_property(dialect_libs GLOBAL PROPERTY GC_DIALECT_LIBS)
   get_property(conversion_libs GLOBAL PROPERTY GC_PASS_LIBS)
+  get_property(extension_libs GLOBAL PROPERTY GC_EXTENSION_LIBS)
 else()
   set(MLIR_LINK_COMPONENTS
     MLIROptLib
+    MLIRToLLVMIRTranslationRegistration
   )
   get_property(dialect_libs GLOBAL PROPERTY MLIR_DIALECT_LIBS)
   get_property(conversion_libs GLOBAL PROPERTY MLIR_CONVERSION_LIBS)
+  get_property(extension_libs GLOBAL PROPERTY MLIR_EXTENSION_LIBS)
 endif()
 
 add_llvm_executable(gc-opt gc-opt.cpp)

src/gc-opt/gc-opt.cpp

@@ -67,7 +67,11 @@ int main(int argc, char *argv[]) {
   registry.insert<mlir::microkernel::MicrokernelDialect>();
   registry.insert<mlir::gen::GENDialect>();
   mlir::registerAllDialects(registry);
+  // covers lowerings for weird dialects like ub
+  // TODO: avoid `registerALL` to remove this
   mlir::registerAllExtensions(registry);
+  // Adds missing `LLVMTranslationDialectInterface` registration for dialect for
+  // gpu.module op
   mlir::registerAllToLLVMIRTranslations(registry);
   mlir::gen::registerGenTargetInterfaceExternalModels(registry);
   mlir::registerGENDialectTranslation(registry);