[PATCH] D52212: [LLVM-C][OCaml] Add UnifyFunctionExitNodes pass to C and OCaml APIs

Josh Berdine via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 18 04:37:32 PDT 2018


jberdine added inline comments.


================
Comment at: bindings/ocaml/transforms/scalar_opts/llvm_scalar_opts.ml:124
   = "llvm_add_basic_alias_analysis"
+external add_merge_return
+  : [< Llvm.PassManager.any ] Llvm.PassManager.t -> unit
----------------
whitequark wrote:
> Shouldn't this function be called the same way it's called in C API and `opt -help`? Otherwise it's confusing.
Will change. It wasn't clear to me, since `opt -help` has:
```
    -mergereturn                                    - Unify function exit nodes
```



Repository:
  rL LLVM

https://reviews.llvm.org/D52212





More information about the llvm-commits mailing list