================
@@ -1000,6 +1019,19 @@ bool Target::ProcessIsValid() {
return (m_process_sp && m_process_sp->IsAlive());
}
+lldb::BreakpointResolverSP
+Target::CheckBreakpointOverrides(lldb::BreakpointResolverSP original_sp) {
+ for (auto const &elem : m_breakpoint_overrides) {
+ if (!original_sp->ResolverTyInMask(elem.second->GetTypeMask()))
+ continue;
+ lldb::BreakpointResolverSP overriden_sp;
+ overriden_sp = elem.second->CheckForOverride(*this, original_sp);
+ if (overriden_sp)
+ return overriden_sp;
----------------
JDevlieghere wrote:
This is 3 line for something that can be done in one. At least it should
collapse the default construction with the copy assignment. But even better
just do:
```suggestion
if (lldb::BreakpointResolverSP overriden_sp =
elem.second->CheckForOverride(*this, original_sp))
return overriden_sp;
```
https://github.com/llvm/llvm-project/pull/198845
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
