[Lldb-commits] [PATCH] D30520: Make LLDB skip server-client roundtrip for signals that don't require any actions
Eugene Zemtsov via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Mar 6 16:32:47 PST 2017
eugene added inline comments.
Comment at: include/lldb/Target/Process.h:3148
+ virtual Error UpdateAutomaticSignalFiltering();
> Can we remove this and only have it in ProcessGDBRemote? Then we just call it when we need to in ProcessGDBRemote? This seems like something that ProcessGDBRemote should take care of without having Process knowing about it. Seems like a adding a call to ProcessGDBRemote::UpdateAutomaticSignalFiltering() can be done in ProcessGDBRemote::WillResume() and ProcessGDBRemote::DidLaunch() might do the trick? It seems like each process plug-in will have different needs regarding signals.
I 100% agree with you here. UpdateAutomaticSignalFiltering() is very much implementation specific and it shouldn't be in the lldb_private::Process. And it was how I originally implemented it in older revision.
But Jim felt differently and I trusted his judgement given that he has vastly more experience working with LLDB codebase.
More information about the lldb-commits