search

facebookincubator / dynolog

Dynolog is a telemetry daemon for performance monitoring and tracing. It exports metrics from different components in the system like the linux kernel, CPU, disks, Intel PT, GPUs etc. Dynolog also integrates with pytorch and can trigger traces for distributed training applications.
MIT License
187 stars 34 forks source link

Remove unused variables in gloo2/test/alltoallv_cpu_test.cc #260

Closed r-barnes closed 1 month ago

r-barnes commented 1 month ago

Summary: LLVM-15 has a warning -Wunused-but-set-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]], mostly in cases where the variable is used, but, eg, in an assert statement that isn't present in production code.

Differential Revision: D57343994

facebook-github-bot commented 1 month ago

This pull request was exported from Phabricator. Differential Revision: D57343994

facebook-github-bot commented 1 month ago

This pull request has been merged in facebookincubator/dynolog@39a69707b30526baf88fceeb936cac8e6e2eda82.