There have been a few suggestions that I could just use a script to solve
this "problem" -- poor startup performance of clangdiag.
However, this patch was not submitted to solve a particular problem. It
was submitted in response to Jim's suggestion:
On Mon, Oct 23, 2017 at 6:25 PM, Jim Ingham <jing...@apple.com> wrote:
> Yeah, that would be easy to implement from the command line, maybe add a
> --file-is-regex flag or something.
>
> From the SB API it would be better to have something like:
>
> SBFileList SBTarget.GetFileListMatchingRegex("regex")
>
> Please file an enhancement request for these of hack'em in if you're so
> motivated.
>
clangdiag was only provided as a motivating example.
As for the contention that this would be a little used option, I'd argue
the opposite.
I use regular expressions all the time, and I'm sure I'm not alone. Once
uses are given the choice of providing multiple '-f' and/or '-s' options
compare to a single, more powerful '-z' or '-Z' option, I'd bet most of
them would start using the new options exclusively. I certainly would.
On Mon, Oct 30, 2017 at 10:03 PM, Don Hinton <hinto...@gmail.com> wrote:
> I'll add tests if it looks like it'll be accepted, but based on the
> initial response, that doesn't seem likely.
>
> However, it was a good exercise and addressed the issues raised.
>
> thanks again for all the feedback...
> don
>
> On Mon, Oct 30, 2017 at 9:44 PM, Zachary Turner <ztur...@google.com>
> wrote:
>
>> Asking again, but why can’t this be done in th the script for clangdiag?
>> For example, there’s no tests for any of this in this patch. And it seems
>> likely that it will be rarely used anyway. So I’m still not convinced the
>> option-pollution and increased long term code maintenance burden of this
>> underutilized codepath is worth the benefit.
>>
>> Can you see if manually scanning for these files in python and then
>> setting breakpoints on the right set of files solves the problem?
>>
>> On Mon, Oct 30, 2017 at 9:00 PM Don Hinton via Phabricator <
>> revi...@reviews.llvm.org> wrote:
>>
>>> hintonda updated this revision to Diff 120933.
>>> hintonda added a comment.
>>>
>>> - Remove prefix and add options.
>>>
>>>
>>> https://reviews.llvm.org/D39436
>>>
>>> Files:
>>> include/lldb/Utility/FileSpec.h
>>> source/Commands/CommandObjectBreakpoint.cpp
>>> source/Utility/FileSpec.cpp
>>>
>>>
>
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
