commit 4872b126fdd357b6e4d9aa7db1b82466a8750966
author Jongsoo Park <jongsoo@fb.com> Fri Mar 06 09:45:55 2020 -0800
committer Facebook Github Bot <facebook-github-bot@users.noreply.github.com> Fri Mar 06 09:52:43 2020 -0800
tree 5a9419dc893b504af3a4f29255ebeaba724111e3
parent 82a177c07fc30a6d9918d16b77e56f2d37963d14

[aten] remove stmt unreachable, variable never used warnings (#34017)

Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/34017

Remove warning
```
caffe2/aten/src/THC/generic/THCTensorMathBlas.cu(437): warning: statement is unreachable
caffe2/aten/src/THC/generic/THCTensorMathBlas.cu(271): warning: variable "transpose_m1" was set but never used
caffe2/aten/src/THC/generic/THCTensorMathBlas.cu(271): warning: variable "transpose_m2" was set but never used
```

Test Plan: CI

Reviewed By: ngimel

Differential Revision: D20181179

fbshipit-source-id: 3665912ba55bffbd8b4555f8a6803e57a502c103

aten/src/THC/generic/THCTensorMathBlas.cu

diff --git a/aten/src/THC/generic/THCTensorMathBlas.cu b/aten/src/THC/generic/THCTensorMathBlas.cu
index bb2546d..d276365 100644
--- a/aten/src/THC/generic/THCTensorMathBlas.cu
+++ b/aten/src/THC/generic/THCTensorMathBlas.cu
@@ -413,8 +413,6 @@
                    beta,
                    THCTensor_(data)(state, r__),
                    r__->stride((transpose_r == 'n' ? 1 : 0)));
-#else
-  TORCH_CHECK(false, "Bgemm not supported on at::BFloat16 type");
 #endif // __HIP_PLATFORM_HCC__
 #elif defined(THC_REAL_IS_DOUBLE)
   THCudaBlas_Dgemm(state,
@@ -445,6 +443,12 @@
   if(r__ != r_) {
     THCTensor_(freeCopyTo)(state, r__, r_);
   }
+
+#if defined(THC_REAL_IS_BFLOAT16) && !defined(__HIP_PLATFORM_HCC__)
+  // To avoid "variable was set but never used" warning
+  [&transpose_m1, &transpose_m2]{}();
+  TORCH_CHECK(false, "Bgemm not supported on at::BFloat16 type");
+#endif
 #else
   ERROR_ONLY_FP_TYPES("addmm");
 #endif
@@ -793,8 +797,6 @@
       beta,
       THCTensor_(data)(state, result_), ldc, result_->stride(0),
       num_batches);
-#else
-  TORCH_CHECK(false, "BgemmStridedBatched is not supported with at::BFloat16 type");
 #endif // __HIP_PLATFORM_HCC__
 #endif
 
@@ -809,8 +811,16 @@
   if (result_ != result) {
     THCTensor_(freeCopyTo)(state, result_, result);
   }
+
+#if defined(THC_REAL_IS_BFLOAT16) && !defined(__HIP_PLATFORM_HCC__)
+  // To avoid "variable was set but never used" warning
+  [&transpose_batch1, &transpose_batch2, &lda, &ldb, &ldc]{}();
+  TORCH_CHECK(false, "BgemmStridedBatched is not supported with at::BFloat16 type");
+#endif
   }
+#if !defined(THC_REAL_IS_BFLOAT16) || defined(__HIP_PLATFORM_HCC__)
   at::namedinference::propagate_names_if_nonempty(result, maybe_outnames);
+#endif
 
 #else
   ERROR_ONLY_FP_TYPES("baddbmm");