Enable clang-tidy on aten/src/ATen/cpu (#132830)

Expands code coverage of clang-tidy to aten/src/ATen/cpu

Pull Request resolved: https://github.com/pytorch/pytorch/pull/132830
Approved by: https://github.com/Skylion007
This commit is contained in:
cyy 2024-08-07 14:44:17 +00:00 committed by PyTorch MergeBot
parent ed97fb77f9
commit 13fa59580e
6 changed files with 14 additions and 13 deletions

View file

@ -195,6 +195,8 @@ include_patterns = [
# and excluding most sub-directories for now.
'aten/src/ATen/*.h',
'aten/src/ATen/*.cpp',
'aten/src/ATen/cpu/*.h',
'aten/src/ATen/cpu/*.cpp',
'aten/src/ATen/core/*.h',
'aten/src/ATen/core/*.cpp',
'aten/src/ATen/cudnn/*.h',

View file

@ -85,7 +85,7 @@ static uint32_t get_cache_size(int level) {
if (!processors) {
return 0;
}
const struct cpuinfo_cache* cache = NULL;
const struct cpuinfo_cache* cache = nullptr;
switch (level) {
case 1:
cache = processors[0].cache.l1d;

View file

@ -15,7 +15,7 @@ inline Vectorized<bool> convert_to_bool(Vectorized<int8_t> x) {
x.ne(Vectorized<int8_t>(0)).store(buffer);
Vectorized<bool> ret;
static_assert(x.size() == ret.size(), "");
static_assert(x.size() == ret.size());
std::memcpy(ret, buffer, ret.size() * sizeof(bool));
return ret;
}

View file

@ -848,22 +848,22 @@ static inline Vectorized<T> bitwise_binary_op(const Vectorized<T> &a, const Vect
return Vectorized<T>::loadu(buffer);
}
template<class T, typename std::enable_if_t<!std::is_base_of<Vectorizedi, Vectorized<T>>::value, int> = 0>
template<class T, typename std::enable_if_t<!std::is_base_of_v<Vectorizedi, Vectorized<T>>, int> = 0>
inline Vectorized<T> operator&(const Vectorized<T>& a, const Vectorized<T>& b) {
return bitwise_binary_op(a, b, std::bit_and<intmax_t>());
}
template<class T, typename std::enable_if_t<!std::is_base_of<Vectorizedi, Vectorized<T>>::value, int> = 0>
template<class T, typename std::enable_if_t<!std::is_base_of_v<Vectorizedi, Vectorized<T>>, int> = 0>
inline Vectorized<T> operator|(const Vectorized<T>& a, const Vectorized<T>& b) {
return bitwise_binary_op(a, b, std::bit_or<intmax_t>());
}
template<class T, typename std::enable_if_t<!std::is_base_of<Vectorizedi, Vectorized<T>>::value, int> = 0>
template<class T, typename std::enable_if_t<!std::is_base_of_v<Vectorizedi, Vectorized<T>>, int> = 0>
inline Vectorized<T> operator^(const Vectorized<T>& a, const Vectorized<T>& b) {
return bitwise_binary_op(a, b, std::bit_xor<intmax_t>());
}
#endif // defined(CPU_CAPABILITY_AVX2) || defined(CPU_CAPABILITY_AVX512)
template<class T, typename std::enable_if_t<!std::is_base_of<Vectorizedi, Vectorized<T>>::value, int> = 0>
template<class T, typename std::enable_if_t<!std::is_base_of_v<Vectorizedi, Vectorized<T>>, int> = 0>
inline Vectorized<T> operator~(const Vectorized<T>& a) {
using int_t = int_same_size_t<T>;
Vectorized<T> ones(c10::bit_cast<T>((int_t)(~(int_t)0))); // All bits are 1

View file

@ -28,13 +28,13 @@ struct VecConvert {
};
template <typename dst_t, typename src_t>
inline typename std::enable_if<std::is_same<dst_t, src_t>::value, Vectorized<src_t>>::type
inline std::enable_if_t<std::is_same_v<dst_t, src_t>, Vectorized<src_t>>
convert(const Vectorized<src_t>& src) {
return src;
}
template <typename dst_t, typename src_t>
inline typename std::enable_if<!std::is_same<dst_t, src_t>::value, Vectorized<dst_t>>::type
inline std::enable_if_t<!std::is_same_v<dst_t, src_t>, Vectorized<dst_t>>
convert(const Vectorized<src_t>& src) {
return VecConvert<dst_t, 1, src_t, 1>::apply(src);
}
@ -56,7 +56,7 @@ template <
int src_n,
bool keep = false,
std::enable_if_t<dst_n == 1, int> = 0>
inline typename std::conditional<keep, VectorizedN<dst_t, 1>, Vectorized<dst_t>>::type
inline std::conditional_t<keep, VectorizedN<dst_t, 1>, Vectorized<dst_t>>
convert(const VectorizedN<src_t, src_n>& src) {
return VecConvert<dst_t, dst_n, src_t, src_n>::apply(src);
}

View file

@ -35,8 +35,8 @@
#include <mkl.h>
#endif
namespace at {
namespace vml {
namespace at::vml {
inline namespace CPU_CAPABILITY {
using namespace vec;
@ -167,5 +167,4 @@ IMPLEMENT_VML_MKL(log2, Log2)
#endif
} // namespace
} // namespace vml
} // namespace at
} // namespace at::vml