johannes abandoned this revision.
johannes added a comment.

split up in several commits starting from https://reviews.llvm.org/D36176



================
Comment at: lib/Tooling/ASTDiff/ASTDiff.cpp:170
+  // Ignore everything from other files.
+  if (!SrcMgr.isInMainFile(SLoc))
+    return true;
----------------
arphaman wrote:
> How will you deal with changes in headers if you limit analysis to main files 
> only? 
I will change this to support headers in another patch


https://reviews.llvm.org/D34748



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D34748: [cla... Johannes Altmanninger via Phabricator via cfe-commits
    • [PATCH] D34748:... Alex Lorenz via Phabricator via cfe-commits
    • [PATCH] D34748:... Johannes Altmanninger via Phabricator via cfe-commits

Reply via email to