That seems reasonable.  Seems Aaron ran into this not because he was trying to do a regex search, but because he *wasnt* trying to do a regex search.  So if he doesn’t have immediate need for a regex search, and if it’s not tested anyway, removing it seems fine <br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 20, 2017 at 10:49 AM Greg Clayton via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg added a comment.<br>
<br>
If you load a exe file that has a PDB file, people can currently run:<br>
<br>
  (lldb) type lookup "char*"<br>
<br>
If no testing is using the regular expression stuff, then just pull it out.  Do we have unit tests that depend on this working? If not, lets just pull it out from the SymbolFilePDB::FindTypes() and we can leave the fix in that uses the RegularExpression class so it won't crash in the future?<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D41086" rel="noreferrer" target="_blank">https://reviews.llvm.org/D41086</a><br>
<br>
<br>
<br>
</blockquote></div>