[PATCH] D52210: [LLVM-C] Add C APIs to access DebugLoc info

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 10 08:53:40 PDT 2018


compnerd accepted this revision.
compnerd added a comment.

This seems reasonable to me.  @whitequark, are you okay with them too?  To @whitequark's original point, I think that the dispatching the type based on a query is pretty cool (you can do an enumeration + union output), but, I think API-wise, this is safer and easier to work with, not to mention also simplifies the implementation.


Repository:
  rL LLVM

https://reviews.llvm.org/D52210





More information about the llvm-commits mailing list