Fix a number of warnings when building TF on Mac.

Mostly unused member variables.

PiperOrigin-RevId: 231259867

tensorflow/core/grappler/mutable_graph_view.cc

@@ -52,10 +52,6 @@ bool IsTensorIdControlling(const TensorId& tensor_id) {
   return tensor_id.index() == Graph::kControlSlot;
 }
 
-bool IsOutputPortRegular(const MutableGraphView::OutputPort& port) {
-  return port.port_id > Graph::kControlSlot;
-}
-
 bool IsOutputPortControlling(const MutableGraphView::OutputPort& port) {
   return port.port_id == Graph::kControlSlot;
 }

tensorflow/core/grappler/optimizers/dependency_optimizer.h

@@ -30,8 +30,7 @@ namespace grappler {
 class DependencyOptimizer : public GraphOptimizer {
  public:
   DependencyOptimizer() {}
-  explicit DependencyOptimizer(RewriterConfig::Toggle opt_level)
+  explicit DependencyOptimizer(RewriterConfig::Toggle opt_level) {}
-      : opt_level_(opt_level) {}
   ~DependencyOptimizer() override {}
 
   string name() const override { return "dependency_optimizer"; };
@@ -73,7 +72,6 @@ class DependencyOptimizer : public GraphOptimizer {
   // single control edge.
   void GroupCrossDeviceControlEdges();
 
-  RewriterConfig::Toggle opt_level_;
   bool fetch_nodes_known_;
   std::unordered_set<string> nodes_to_preserve_;
   std::unique_ptr<NodeMap> node_map_;

tensorflow/core/grappler/optimizers/pin_to_host_optimizer.h

@@ -38,9 +38,8 @@ string TryFindHostDevice(const gtl::FlatSet<string>& devices,
 // gpu->gpu->gpu may have been better/faster. We should probably fix this.
 class PinToHostOptimizer : public GraphOptimizer {
  public:
-  PinToHostOptimizer() : opt_level_(RewriterConfig::DEFAULT) {}
+  PinToHostOptimizer() {}
-  explicit PinToHostOptimizer(RewriterConfig::Toggle opt_level)
+  explicit PinToHostOptimizer(RewriterConfig::Toggle opt_level) {}
-      : opt_level_(opt_level) {}
 
   ~PinToHostOptimizer() override {}
 
@@ -51,9 +50,6 @@ class PinToHostOptimizer : public GraphOptimizer {
 
   void Feedback(Cluster* cluster, const GrapplerItem& item,
                 const GraphDef& optimized_graph, double result) override {}
-
- private:
-  RewriterConfig::Toggle opt_level_;
 };
 
 }  // end namespace grappler

tensorflow/core/grappler/optimizers/remapper.h

@@ -26,7 +26,7 @@ namespace grappler {
 // nodes to decrease the amount of operations needed to perform a computation.
 class Remapper : public GraphOptimizer {
  public:
-  explicit Remapper(RewriterConfig::Toggle opt_level) : opt_level_(opt_level) {}
+  explicit Remapper(RewriterConfig::Toggle opt_level) {}
 
   ~Remapper() override {}
 
@@ -37,9 +37,6 @@ class Remapper : public GraphOptimizer {
 
   void Feedback(Cluster* cluster, const GrapplerItem& item,
                 const GraphDef& optimized_graph, double result) override;
-
- private:
-  RewriterConfig::Toggle opt_level_;
 };
 
 }  // end namespace grappler

tensorflow/core/grappler/optimizers/shape_optimizer.h

@@ -30,9 +30,8 @@ namespace grappler {
 // information.
 class ShapeOptimizer : public GraphOptimizer {
  public:
-  ShapeOptimizer() : opt_level_(RewriterConfig::ON) {}
+  ShapeOptimizer() {}
-  explicit ShapeOptimizer(RewriterConfig::Toggle opt_level)
+  explicit ShapeOptimizer(RewriterConfig::Toggle opt_level) {}
-      : opt_level_(opt_level) {}
 
   ~ShapeOptimizer() override {}
 
@@ -43,9 +42,6 @@ class ShapeOptimizer : public GraphOptimizer {
 
   void Feedback(Cluster* cluster, const GrapplerItem& item,
                 const GraphDef& optimized_graph, double result) override;
-
- private:
-  RewriterConfig::Toggle opt_level_;
 };
 
 }  // end namespace grappler

tensorflow/core/platform/posix/env.cc

@@ -86,7 +86,7 @@ class PosixEnv : public Env {
     return new StdThread(thread_options, name, fn);
   }
 
-  int32 GetCurrentThreadId() {
+  int32 GetCurrentThreadId() override {
 #ifdef __APPLE__
     uint64_t tid64;
     pthread_threadid_np(nullptr, &tid64);
@@ -101,7 +101,7 @@ class PosixEnv : public Env {
 #endif
   }
 
-  bool GetCurrentThreadName(string* name) {
+  bool GetCurrentThreadName(string* name) override {
 #ifdef __ANDROID__
     return false;
 #else

tensorflow/core/util/strided_slice_op.cc

@@ -272,7 +272,7 @@ Status ValidateStridedSliceOp(
     const std::array<int64, 2> valid_range = {
         {stride_i > 0 ? 0 : -1, stride_i > 0 ? dim_i : dim_i - 1}};
 
-    auto canonical = [stride_i, i, dim_i, masks, valid_range](int64 x, int c) {
+    auto canonical = [stride_i, dim_i, masks, valid_range](int64 x, int c) {
       if (masks[c]) {
         return stride_i > 0 ? valid_range[c] : valid_range[(c + 1) & 1];
       } else {