Skip to content

Commit 34cdb1d

Browse files
r-barnesfacebook-github-bot
authored andcommitted
Remove unused-variable in tiles/test/TilesTestBase.cpp +3 (#2572)
Summary: Pull Request resolved: #2572 LLVM-15 has a warning `-Wunused-variable` which we treat as an error because it's so often diagnostic of a code issue. Unused variables can compromise readability or, worse, performance. This diff either (a) removes an unused variable and, possibly, it's associated code or (b) qualifies the variable with `[[maybe_unused]]`. #buildsonlynotests - Builds are sufficient - If you approve of this diff, please use the "Accept & Ship" button :-) Reviewed By: wuyuoss Differential Revision: D66143538 fbshipit-source-id: 18bf044ca416517b78669f86cc43f9ba0a6e3964
1 parent 9cc85d9 commit 34cdb1d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

torchrec/inference/inference_legacy/src/SingleGPUExecutor.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ SingleGPUExecutor::SingleGPUExecutor(
3838
for (size_t i = 0; i < numProcessThreads_; ++i) {
3939
processExecutor_->add([&]() { process(); });
4040
}
41-
for (const auto& exec_info : execInfos_) {
41+
for ([[maybe_unused]] const auto& exec_info : execInfos_) {
4242
TORCHREC_CHECK(exec_info.interpIdx < manager_->allInstances().size());
4343
}
4444
TORCHREC_CHECK(observer_);

0 commit comments

Comments
 (0)