[llvm] [LLVM-C] Add bindings to `Instruction::getDbgRecordRange()` (PR #107802)

Michal Rostecki via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 18 08:39:47 PDT 2024


vadorovsky wrote:

@OCHyams Done.

I don't mind these nitpicks, but I just wanted to mention that many of the nitpicked sentences, like:

* "Advance a [...] iterator"
* "Go backwards in [...] iterator"
* "Obtain the next [...] in [...]"

are not my invention and there were already used on other docstrings (e.g. `LLVMGet*BasicBlock`). I think it would be great to adjust these now too, to avoid inconsistency. Not sure if I should submit a (separate) PR or you would prefer to do that? I don't mind either way.

https://github.com/llvm/llvm-project/pull/107802


More information about the llvm-commits mailing list