[PATCH] D41356: Don't look for OCaml if LLVM_ENABLE_OCAMLDOC=OFF

whitequark via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 19 03:19:13 PST 2017


whitequark added a comment.

> how could OCaml bindings be turned off in LLVM if OCaml is installed at configure time? (and avoiding the package lookup if it's not needed?)

That's not supported for any of the bindings right now.

> Another comment is that include(FindOCaml) and find_package(OCaml) serve more or less the same purpose, so the error message appears twice. One of the lines could be removed.

ACK on this.


Repository:
  rL LLVM

https://reviews.llvm.org/D41356





More information about the llvm-commits mailing list