[PATCH] D27464: Use the fallible llvm::Regex constructor in Clang.

David L. Jones via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 7 14:49:05 PST 2016


dlj updated this revision to Diff 80670.
dlj added a comment.

- Added comments about dropped assertion checks, and ensured checks in other places.
- (Probably?) Fixed a unit test. The intent isn't clear, it's an un-handleable erroneous input case.


https://reviews.llvm.org/D27464

Files:
  include/clang/ASTMatchers/ASTMatchers.h
  include/clang/Frontend/VerifyDiagnosticConsumer.h
  lib/Driver/Multilib.cpp
  lib/Format/ContinuationIndenter.cpp
  lib/Format/Format.cpp
  lib/Format/FormatTokenLexer.cpp
  lib/Frontend/CompilerInvocation.cpp
  lib/Frontend/VerifyDiagnosticConsumer.cpp
  tools/driver/driver.cpp
  unittests/ASTMatchers/ASTMatchersNodeTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27464.80670.patch
Type: text/x-patch
Size: 11466 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161207/06ca9bdd/attachment.bin>


More information about the llvm-commits mailing list