[Lldb-commits] [PATCH] D116788: [lldb] Set result error state in 'frame variable'
Dave Lee via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Jan 10 10:17:32 PST 2022
kastiglione added a comment.
In D116788#3231612 <https://reviews.llvm.org/D116788#3231612>, @clayborg wrote:
> I would vote that if any argument or option is specified, and results in something not being found, that an error be returned.
Is this in response to my question about regexes? It seems to be. Specifically, you're of the opinion that any failing regex results in an error, even when other regexes do match? Your point about it not affecting the command line is a good point, thanks.
All your other points sound good to me too.
> Any tests that are failing now when they didn't before can be modified.
Yes there were a couple, D116901 <https://reviews.llvm.org/D116901> and D116863 <https://reviews.llvm.org/D116863>.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116788/new/
https://reviews.llvm.org/D116788
More information about the lldb-commits
mailing list