Re: [PATCH] D19877: [clang-tidy] Speedup misc-static-assert.

2016-05-04 Thread Alexander Kornienko via cfe-commits
alexfh added a comment. Thank you for finding and fixing this! I think I used to suggest the wrong pattern, now I won't. Repository: rL LLVM http://reviews.llvm.org/D19877 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm

Re: [PATCH] D19877: [clang-tidy] Speedup misc-static-assert.

2016-05-03 Thread Samuel Benzaquen via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL268430: [clang-tidy] Speedup misc-static-assert. (authored by sbenza). Changed prior to commit: http://reviews.llvm.org/D19877?vs=56028&id=56053#toc Repository: rL LLVM http://reviews.llvm.org/D1987

Re: [PATCH] D19877: [clang-tidy] Speedup misc-static-assert.

2016-05-03 Thread Aaron Ballman via cfe-commits
aaron.ballman added a subscriber: aaron.ballman. aaron.ballman accepted this revision. aaron.ballman added a reviewer: aaron.ballman. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM, thank you for noticing that! http://reviews.llvm.org/D19877 _