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

Author: kurapov-peter

src/gc-opt/CMakeLists.txt

@@ -21,17 +21,21 @@ 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()
 
 set(gc_opt_libs 
         ${dialect_libs}
         ${conversion_libs} 
+        ${extension_libs}
         ${MLIR_LINK_COMPONENTS}
         GCPasses
         GCGPUPasses)

src/gc-opt/gc-opt.cpp

@@ -61,7 +61,11 @@ int main(int argc, char *argv[]) {
   registry.insert<mlir::linalgx::LinalgxDialect>();
   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);