[PATCH] D60484: [LLVM-C] Correct The Current Debug Location Accessors
whitequark via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 9 15:11:33 PDT 2019
whitequark added inline comments.
================
Comment at: llvm/include/llvm-c/Core.h:3531
+
+void LLVMSetInstDebugLocation(LLVMBuilderRef Builder, LLVMValueRef Inst);
+/**
----------------
CodaFi wrote:
> whitequark wrote:
> > Let's add the `2` suffix here for consistency as well.
> This API didn't change, the diff is just a mess!
Oh... Can you add some docs for this API? I had to look up the C++ source to even understand what it's supposed to do.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60484/new/
https://reviews.llvm.org/D60484
More information about the llvm-commits
mailing list