On Tue, Oct 31, 2017 at 8:22 AM, Zachary Turner <ztur...@google.com> wrote:
> > > On Tue, Oct 31, 2017 at 8:12 AM Don Hinton <hinto...@gmail.com> wrote: > >> 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. >> > > Fair point, I had forgotten about that. That said, I'm honestly still not > that big of a fan. It's no secret that I have a higher bar than others > for options and API additions, but to me the potential use this particular > option + SB API would get is not sufficient to justify the long term > maintenance burden associated with having it. > I'm not that familiar with the SB API part, but will investigate, and try to come up with a better motivation. Thanks again for the suggestions and feedback...
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits