[PATCH] D80760: [JitRunner] add support for i32 output

[Uday Bondhugula via Phabricator via llvm-commits]

bondhugula requested changes to this revision.
bondhugula added inline comments.
This revision now requires changes to proceed.


================
Comment at: mlir/lib/ExecutionEngine/JitRunner.cpp:56
       "entry-point-result",
       llvm::cl::desc("Textual description of the function type to be called"),
+      llvm::cl::value_desc("f32 | i32 | void"), llvm::cl::init("f32")};
----------------
-> "Function result type" ?


================
Comment at: mlir/lib/ExecutionEngine/JitRunner.cpp:215
+
+  int res;
+  struct {
----------------
Should this be `int32_t`?


================
Comment at: mlir/lib/ExecutionEngine/JitRunner.cpp:297-302
+<<<<<<< HEAD:mlir/lib/ExecutionEngine/JitRunner.cpp
       llvm::StringSwitch<CompileAndExecuteFnT>(options.mainFuncType.getValue())
+=======
+      llvm::StringSwitch<CompileAndExecuteFnT>(mainFuncType.getValue())
+          .Case("i32", compileAndExecuteSingleIntReturnFunction)
+>>>>>>> 124f2402e0d... [JitRunner] add support for i32 output:mlir/lib/Support/JitRunner.cpp
----------------
Conflict markers. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80760/new/

https://reviews.llvm.org/D80760