[Lldb-commits] [PATCH] D51453: Refactor BreakpointResolver::SetSCMatchesByLine()

[Jim Ingham via Phabricator via lldb-commits]

jingham added a comment.

By name breakpoints don't use this function so a function regex wouldn't show the change.  But the source regex does use this filter, so something like:

(lldb) break set -p ;

on a large source file might do it.  That's a pretty silly thing to do, however.  The only actual case where this might matter is if you have code that gets inlined all over the place and you set a breakpoint on the inlined code.  But IRL I don't think the performance part of this change is really going to matter much.


https://reviews.llvm.org/D51453