[PATCH] D52239: [OCaml] Add OCaml APIs to access DebugLoc info

[whitequark via Phabricator via llvm-commits]

whitequark added inline comments.


================
Comment at: bindings/ocaml/llvm/llvm_ocaml.c:873
+  if ((Chars = LLVMGetDebugLocDirectory(Val, &Length))) {
+    CAMLlocal2(Option, String);
+    String = caml_alloc_string(Length);
----------------
Is it legal to use `CAMLlocal2` in a conditional like this?


Repository:
  rL LLVM

https://reviews.llvm.org/D52239