Commit 4070855d authored by Paul's avatar Paul
Browse files

Fix tidy warnings

parent a6ea39ea
...@@ -470,9 +470,9 @@ inline auto name(std::unordered_set<std::string> names) ...@@ -470,9 +470,9 @@ inline auto name(std::unordered_set<std::string> names)
} }
template <class... Ts> template <class... Ts>
inline auto name(std::string s, Ts... xs) inline auto name(std::string s, Ts... xs) // NOLINT
{ {
return name(std::unordered_set<std::string>{s, xs...}); return name(std::unordered_set<std::string>{std::move(s), std::move(xs)...});
} }
inline auto nargs(std::size_t n) inline auto nargs(std::size_t n)
......
...@@ -283,7 +283,7 @@ auto nary_standard(hipStream_t stream, argument result, Arguments... args) ...@@ -283,7 +283,7 @@ auto nary_standard(hipStream_t stream, argument result, Arguments... args)
template <class... Arguments> template <class... Arguments>
bool broadcastable( bool broadcastable(
bool& divisible_by_4, std::size_t max_size, argument result, argument barg, Arguments... args) bool& divisible_by_4, std::size_t max_size, const argument& result, const argument& barg, const Arguments&... args)
{ {
divisible_by_4 = false; divisible_by_4 = false;
auto bshape = barg.get_shape(); auto bshape = barg.get_shape();
...@@ -312,7 +312,7 @@ bool broadcastable( ...@@ -312,7 +312,7 @@ bool broadcastable(
return false; return false;
} }
inline bool broadcastable(bool& divisible_by_4, std::size_t, argument, argument) inline bool broadcastable(bool& divisible_by_4, std::size_t, const argument&, const argument&)
{ {
divisible_by_4 = false; divisible_by_4 = false;
return false; return false;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment