[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