Finish up the episode 9

docs/videos.org

@@ -377,13 +377,19 @@ define i64 @main1(i64 %0, i64 %1, i64 %2) !dbg !9 {
 - https://mlir.llvm.org/docs/LangRef
 - https://en.wikipedia.org/wiki/Basic_block
 
-* Episode 9 - Code Generation
+* Episode 9 - IR (SLIR) generation
 ** Updates:
-There will be an episode dedicated to eache of these
-
 - Source manager
 - Diagnostic Engine
 - JIT
+
+There will be an episode dedicated to eache of these
+** How does IR generation works
+- Pass around MLIR context
+- Create Builder objects that creates operations in specific
+  locations
+- ModuleOp
+- Namespace
 ** How to define a new dialect
 - Pure C++
 - Tablegen
@@ -392,7 +398,7 @@ There will be an episode dedicated to eache of these
 - An IR that follows the AST
 - Rename?
 *** Steps
-- [ ] Define the new dialect
-- [ ] Setup the tablegen
-- [ ] Define the operations
-- [ ] Walk the AST and generate the operations
+- [X] Define the new dialect
+- [X] Setup the tablegen
+- [X] Define the operations
+- [X] Walk the AST and generate the operations

include/serene/slir/CMakeLists.txt

@@ -1,6 +1,8 @@
 set(LLVM_TARGET_DEFINITIONS dialect.td)
+
 mlir_tablegen(ops.h.inc -gen-op-decls)
 mlir_tablegen(ops.cpp.inc -gen-op-defs)
 mlir_tablegen(dialect.h.inc -gen-dialect-decls)
 mlir_tablegen(dialect.cpp.inc -gen-dialect-defs)
+
 add_public_tablegen_target(SereneDialectGen)

include/serene/slir/dialect.td

@@ -47,7 +47,7 @@ def ValueOp: Serene_Op<"value"> {
     let arguments = (ins I64Attr:$value);
     let results = (outs I64);
 
-    // let verifier = [{ return serene::sir::verify(*this); }];
+    //let verifier = [{ return serene::sir::verify(*this); }];
 
     let builders = [
         OpBuilder<(ins "int":$value), [{
@@ -80,26 +80,8 @@ def FnOp: Serene_Op<"fn", [
     let regions = (region AnyRegion:$body);
     let results = (outs I64);
 
-  // let builders = [
-  //     OpBuilder<(ins
-  //       "llvm::StringRef":$name, "mlir::FunctionType":$type,
-  //       CArg<"llvm::ArrayRef<mlir::NamedAttribute>", "{}">:$attrs,
-  //       CArg<"llvm::ArrayRef<mlir::DictionaryAttr>", "{}">:$argAttrs)
-  // >];
-
-  // let extraClassDeclaration = [{
-  //   static FnOp create(mlir::Location location, llvm::StringRef name, mlir::FunctionType type,
-  //                      llvm::ArrayRef<mlir::NamedAttribute> attrs = {});
-    // static FnOp create(mlir::Location location, llvm::StringRef name, mlir::FunctionType type,
-    //                    mlir::Operation::dialect_attr_range attrs);
-    // static FnOp create(mlir::Location location, llvm::StringRef name, mlir::FunctionType type,
-    //                    llvm::ArrayRef<mlir::NamedAttribute> attrs,
-    //                    llvm::ArrayRef<mlir::DictionaryAttr> argAttrs);
-    // }];
 }
 
-
-
 def ReturnOp: Serene_Op<"return", [NoSideEffect, HasParent<"FnOp">,
                                    ReturnLike, Terminator]> {
 
@@ -111,18 +93,6 @@ def ReturnOp: Serene_Op<"return", [NoSideEffect, HasParent<"FnOp">,
     let arguments = (ins AnyType:$operand);
     let assemblyFormat =
           [{ attr-dict $operand  `:` type($operand) }];
-
-    // let verifier = [{ return serene::sir::verify(*this); }];
-
-    // let builders = [
-    //     OpBuilder<(ins "int":$value), [{
-    //     // Build from fix 64 bit int
-    //     build(odsBuilder, odsState, odsBuilder.getI64Type(), (uint64_t) value);
-    //     }]>,
-    // ];
 }
 
-
-
-
 #endif // SERENE_DIALECT

src/serene/exprs/fn.cpp

@@ -127,7 +127,6 @@ void Fn::generateIR(serene::Namespace &ns, mlir::ModuleOp &m) {
         argSym->name, mlir::TypeAttr::get(builder.getI64Type())));
   }
 
-  // auto funcType = builder.getFunctionType(arg_types, builder.getI64Type());
   auto fn = builder.create<slir::FnOp>(
       loc, builder.getI64Type(), name,
       mlir::DictionaryAttr::get(builder.getContext(), arguments),

src/serene/namespace.cpp

@@ -111,11 +111,10 @@ void Namespace::dump() {
     return;
   }
 
-  // mlir::OpPrintingFlags flags;
-  // flags.enableDebugInfo();
+  mlir::OpPrintingFlags flags;
+  flags.enableDebugInfo();
 
-  // maybeModuleOp.getValue()->print(llvm::outs(), flags);
-  maybeModuleOp.getValue()->dump();
+  maybeModuleOp.getValue()->print(llvm::outs(), flags);
 };
 
 MaybeModule Namespace::compileToLLVM() {
@@ -134,7 +133,7 @@ MaybeModule Namespace::compileToLLVM() {
   return MaybeModule::error(true);
 };
 
-Namespace::~Namespace(){};
+  Namespace::~Namespace(){};
 
 std::shared_ptr<Namespace>
 makeNamespace(SereneContext &ctx, llvm::StringRef name,