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
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
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
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
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