[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