[PATCH] D35787: [clang-tidy] Ignore vector in inefficient-vector-operation.

2018-01-05 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added a comment. In https://reviews.llvm.org/D35787#830351, @hokein wrote: > In https://reviews.llvm.org/D35787#819007, @alexfh wrote: > > > Should we look at whether the size is statically known? > > > Do you mean the size of the template type T in `vector`? STL only > provides a templa

[PATCH] D35787: [clang-tidy] Ignore vector in inefficient-vector-operation.

2017-08-03 Thread Haojian Wu via Phabricator via cfe-commits
hokein added a comment. In https://reviews.llvm.org/D35787#819007, @alexfh wrote: > Should we look at whether the size is statically known? Do you mean the size of the template type T in `vector`? STL only provides a template specialization for `std::vector`, so I think excluding `vector` is

[PATCH] D35787: [clang-tidy] Ignore vector in inefficient-vector-operation.

2017-07-24 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added a comment. In https://reviews.llvm.org/D35787#819007, @alexfh wrote: > Should we look at whether the size is statically known? (and small enough) https://reviews.llvm.org/D35787 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D35787: [clang-tidy] Ignore vector in inefficient-vector-operation.

2017-07-24 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added a comment. Should we look at whether the size is statically known? https://reviews.llvm.org/D35787 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D35787: [clang-tidy] Ignore vector in inefficient-vector-operation.

2017-07-24 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. Herald added subscribers: xazax.hun, JDevlieghere. https://reviews.llvm.org/D35787 Files: clang-tidy/performance/InefficientVectorOperationCheck.cpp test/clang-tidy/performance-inefficient-vector-operation.cpp Index: test/clang-tidy/performance-inefficient-vect