[Lldb-commits] [PATCH] D125325: Pass plugin_name in SBProcess::SaveCore

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed May 11 09:12:45 PDT 2022


jingham added a comment.

In D125325#3505305 <https://reviews.llvm.org/D125325#3505305>, @PatriosTheGreat wrote:

> Hi Greg.
>
> Thanks for the review.
> I fixed the feedback.
> I forgot the default initialization of plugin_name parameter in SBProcess.i in previous version, but I assume it's still better to explicitly create a separate method.

Yes, a method with a default parameter has a different mangled name from the method w/o the parameter, so it's still a binary incompatible change.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125325/new/

https://reviews.llvm.org/D125325



More information about the lldb-commits mailing list