[Lldb-commits] [PATCH] D66174: [Utility] Reimplement RegularExpression on top of llvm::Regex

[Jim Ingham via Phabricator via lldb-commits]

jingham accepted this revision.
jingham added a comment.

Pavel said "we'll just have to bite the bullet and say that our expressions are now (more or less) POSIX conformant"...   We should say this somewhere users are likely to find.

All the options that take regular expressions look like:

(lldb) help break set
...

  -p <regular-expression> ( --source-pattern-regexp <regular-expression> )

And then for any option value type, you can get help:

(lldb) help regular-expression

  <regular-expression> -- A regular expression.

Maybe it would be good to say a little more here?

With that addition I'm fine with this.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66174/new/

https://reviews.llvm.org/D66174