Re: [PATCH] D12741: [Clang] Fix Clang-tidy misc-use-override warnings, other minor fixes

2015-09-09 Thread Hans Wennborg via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL247218: Fix Clang-tidy misc-use-override warnings, other minor fixes (authored by hans). Changed prior to commit: http://reviews.llvm.org/D12741?vs=34385=34393#toc Repository: rL LLVM

Re: [PATCH] D12741: [Clang] Fix Clang-tidy misc-use-override warnings, other minor fixes

2015-09-09 Thread Hans Wennborg via cfe-commits
hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM http://reviews.llvm.org/D12741 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D12741: [Clang] Fix Clang-tidy misc-use-override warnings, other minor fixes

2015-09-09 Thread Hans Wennborg via cfe-commits
hans added a comment. In http://reviews.llvm.org/D12741#242907, @hans wrote: > It seems this broke the build for some reason. I have reverted it in r247222. > > Example build breakages: > http://lab.llvm.org:8011/builders/perf-x86_64-penryn-O3-polly-fast/builds/11687/steps/compile/logs/stdio, >