[PATCH] D52207: [OCaml] Add OCaml API for LLVMGetIndices

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


jberdine added inline comments.


================
Comment at: bindings/ocaml/llvm/llvm.mli:813
 
+(** [indices i] returns the number of indices for the ExtractValue or
+    InsertValue instruction [i].
----------------
whitequark wrote:
> I think this function returns the indices, not the *number* of indices.
Indeed, nice catch.


Repository:
  rL LLVM

https://reviews.llvm.org/D52207





More information about the llvm-commits mailing list