[PATCH] D19816: [find-all-symbols] Add IWYU private pragma support.

[Benjamin Kramer via cfe-commits]

bkramer added inline comments.

================
Comment at: include-fixer/find-all-symbols/FindAllSymbols.h:16
@@ +15,3 @@
+#include "llvm/ADT/StringRef.h"
+
+#include <map>
----------------
No space between #includes in LLVM.

================
Comment at: include-fixer/find-all-symbols/PragmaCommentHandler.cpp:27
@@ +26,3 @@
+  SmallVector<StringRef, 4> Matches;
+  if (!llvm::Regex(IWYUPragma).match(Text, &Matches))
+    return false;
----------------
Isn't this regex just a complex way to write Text.find(IWYUPragma)?


http://reviews.llvm.org/D19816