FBGEMM icon indicating copy to clipboard operation
FBGEMM copied to clipboard

Remove unused-variable in deeplearning/fbgemm/fbgemm_gpu/src/jagged_tensor_ops/jagged_unique_indices.cu +1

Open q10 opened this issue 9 months ago • 2 comments

Summary: 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]].

  • If you approve of this diff, please use the "Accept & Ship" button :-)

Reviewed By: q10

Differential Revision: D72588935

q10 avatar Apr 08 '25 01:04 q10

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

facebook-github-bot avatar Apr 08 '25 01:04 facebook-github-bot

Deploy Preview for pytorch-fbgemm-docs ready!

Name Link
Latest commit ded0ca2b2d07f8365ed76e95c7d6451a5f771bc9
Latest deploy log https://app.netlify.com/sites/pytorch-fbgemm-docs/deploys/67f479dc8567370008554a20
Deploy Preview https://deploy-preview-3939--pytorch-fbgemm-docs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Apr 08 '25 01:04 netlify[bot]