[flang] Fix warnings

kazutakahirata · kazutakahirata · commit 76c16651c9f0 · 2024-11-04T09:31:29.000-08:00
This patch fixes:

  flang/lib/Semantics/check-omp-structure.cpp:286:27: error: lambda
  capture 'this' is not used [-Werror,-Wunused-lambda-capture]

  flang/lib/Semantics/check-omp-structure.cpp:299:21: error: private
  field 'sctx_' is not used [-Werror,-Wunused-private-field]
diff --git a/flang/lib/Semantics/check-omp-structure.cpp b/flang/lib/Semantics/check-omp-structure.cpp
@@ -283,7 +283,7 @@ struct ContiguousHelper {
     return Visit(x.get());
   }
   template <typename T> std::optional<bool> Visit(const evaluate::Expr<T> &x) {
-    return common::visit([this](auto &&s) { return Visit(s); }, x.u);
+    return common::visit([&](auto &&s) { return Visit(s); }, x.u);
   }
   template <typename T>
   std::optional<bool> Visit(const evaluate::Designator<T> &x) {
@@ -296,7 +296,7 @@ struct ContiguousHelper {
   }
 
 private:
-  SemanticsContext &sctx_;
+  [[maybe_unused]] SemanticsContext &sctx_;
   evaluate::FoldingContext &fctx_;
 };
 } // namespace

Original file line number	Diff line number	Diff line change
`@@ -283,7 +283,7 @@ struct ContiguousHelper {`
`283`	`283`	`return Visit(x.get());`
`284`	`284`	`}`
`285`	`285`	`template <typename T> std::optional<bool> Visit(const evaluate::Expr<T> &x) {`
`286`		`- return common::visit([this](auto &&s) { return Visit(s); }, x.u);`
	`286`	`+ return common::visit([&](auto &&s) { return Visit(s); }, x.u);`
`287`	`287`	`}`
`288`	`288`	`template <typename T>`
`289`	`289`	`std::optional<bool> Visit(const evaluate::Designator<T> &x) {`
`@@ -296,7 +296,7 @@ struct ContiguousHelper {`
`296`	`296`	`}`
`297`	`297`
`298`	`298`	`private:`
`299`		`- SemanticsContext &sctx_;`
	`299`	`+ [[maybe_unused]] SemanticsContext &sctx_;`
`300`	`300`	`evaluate::FoldingContext &fctx_;`
`301`	`301`	`};`
`302`	`302`	`} // namespace`