Skip to content

Commit fa6fef6

Browse files
committed
Satisfy clang-tidy
1 parent 14c4ed2 commit fa6fef6

File tree

2 files changed

+9
-8
lines changed

2 files changed

+9
-8
lines changed

src/LegalizeVectors.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -131,6 +131,7 @@ class ExtractLanes : public IRMutator {
131131

132132
Expr visit(const Shuffle *op) override {
133133
vector<int> new_indices;
134+
new_indices.reserve(lane_count);
134135
for (int i = 0; i < lane_count; ++i) {
135136
new_indices.push_back(op->indices[lane_start + i]);
136137
}
@@ -428,8 +429,8 @@ class LegalizeVectors : public IRMutator {
428429
Expr visit(const Shuffle *op) override {
429430
internal_assert(op->type.lanes() <= max_lanes) << Expr(op);
430431
bool requires_mutation = false;
431-
for (size_t i = 0; i < op->vectors.size(); ++i) {
432-
if (op->vectors[i].type().lanes() > max_lanes) {
432+
for (const auto &vec : op->vectors) {
433+
if (vec.type().lanes() > max_lanes) {
433434
requires_mutation = true;
434435
break;
435436
}

src/Simplify_Shuffle.cpp

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -67,8 +67,8 @@ Expr Simplify::visit(const Shuffle *op, ExprInfo *info) {
6767
arg_used[src_vec_and_lane_idx[i].first] = true;
6868
}
6969
size_t num_args_used = 0;
70-
for (size_t i = 0; i < arg_used.size(); ++i) {
71-
if (arg_used[i]) {
70+
for (bool used : arg_used) {
71+
if (used) {
7272
num_args_used++;
7373
}
7474
}
@@ -83,10 +83,10 @@ Expr Simplify::visit(const Shuffle *op, ExprInfo *info) {
8383
for (int i = 0; i < vi; ++i) {
8484
vector_start_lane += op->vectors[i].type().lanes();
8585
}
86-
for (size_t i = 0; i < new_indices.size(); ++i) {
87-
if (new_indices[i] > vector_start_lane) {
88-
internal_assert(new_indices[i] >= vector_start_lane + lanes_deleted);
89-
new_indices[i] -= lanes_deleted;
86+
for (int &new_index : new_indices) {
87+
if (new_index > vector_start_lane) {
88+
internal_assert(new_index >= vector_start_lane + lanes_deleted);
89+
new_index -= lanes_deleted;
9090
}
9191
}
9292
new_vectors.erase(new_vectors.begin() + vi);

0 commit comments

Comments
 (0)