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

Josh Berdine via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 13 14:09:14 PST 2018


jberdine added a comment.

@CodaFi, sorry for the delayed reply, I've been busy with other things.

I've been trawling commits since I originally implemented this, and based on your question I'm guessing that you see a way in which the `LLVMGetDebugLoc*` functions added in https://reviews.llvm.org/D52210 are (partially) redundant? Is it the case that there is redundancy in the C++ interfaces such that the methods called there can be equivalently expressed in terms of generic metadata and `DILocation`s?

Or are you saying that these additions to the ocaml api are already redundant?


Repository:
  rL LLVM

https://reviews.llvm.org/D52239





More information about the llvm-commits mailing list