[llvm] [RemoveDIs] Add a 'BeforeDbgRecords' parameter to C API isnt insertion functions (PR #92417)

Orlando Cazalet-Hyams via llvm-commits llvm-commits at lists.llvm.org
Mon May 20 07:42:06 PDT 2024


OCHyams wrote:

>> We could add new functions with the extra parameter, keep the existing functions untouched and forward args and a "default" value for the new parameter to the new functions (possibly this is what you're suggesting already)? I guess this would be most "stable" (and I think then we can omit most of the OCaml changes then).

> Yes, that's what I would suggest.

Done, how does this look now?

@alan-j-hu sorry for the turbulence but could you re-review the OCaml changes? They're much smaller this time.

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


More information about the llvm-commits mailing list