[PATCH] D60902: [OCaml] Add OCaml APIs to access DebugInfo

Josh Berdine via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 11 06:49:49 PDT 2019


jberdine added a comment.

@whitequark, with this diff's approach of creating a hierarchy of types to mirror the LLVM-C DI types, is it acceptable to add the types and functions incrementally? That is, could we land this diff and add other types and functions later?

Another question is about opam. Since this diff adds a sub-library, the opam package files will need to change. Is there any experience / best practice about how to handle this? In particular, since the opam package files are not in the llvm repo, it will not be easily possible to pin the llvm dev repo.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60902/new/

https://reviews.llvm.org/D60902





More information about the llvm-commits mailing list