[cfe-commits] r160115 - in /cfe/trunk: include/clang/ASTMatchers/ASTMatchers.h unittests/ASTMatchers/ASTMatchersTest.cpp

Daniel Jasper djasper at google.com
Thu Jul 12 02:23:06 PDT 2012


No real reason. I thought the content of all three changes was closely
related (although no dependencies) and I thought the patch was not getting
too big. But I will keep them separate next time.


On Thu, Jul 12, 2012 at 10:53 AM, Chandler Carruth <chandlerc at google.com>wrote:

> Any reason to combine these patches?
>
> All things being equal, keeping them split up seems useful for reverting
> in the event of regressions or for the sanity of the post-commit reviewers.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120712/5caf6d8f/attachment.html>


More information about the cfe-commits mailing list