Enable indices bounds check by default

Summary: ...

Reviewed By: jianyuh

Differential Revision: D31234815

fbshipit-source-id: 9584e7c00cb85c159522242a9f4c3e1a14f9af55

Author: Andrew Tulloch

fbgemm_gpu/fbgemm_gpu/split_table_batched_embeddings_ops.py

@@ -548,9 +548,8 @@ def forward(
         feature_requires_grad: Optional[Tensor] = None,
     ) -> Tensor:
         (indices, offsets) = indices.long(), offsets.long()
-        # TODO(tulloch): enable this after sufficient forward-compatibility period.
-        # if self.bounds_check_mode_int != BoundsCheckMode.NONE.value:
-        #     torch.ops.fb.bounds_check_indices(self.rows_per_table, indices, offsets, self.bounds_check_mode_int, self.bounds_check_warning)
+        if self.bounds_check_mode_int != BoundsCheckMode.NONE.value:
+            torch.ops.fb.bounds_check_indices(self.rows_per_table, indices, offsets, self.bounds_check_mode_int, self.bounds_check_warning)
         self.step += 1
         if len(self.timesteps_prefetched) == 0:
             self.prefetch(indices, offsets)