[flang-commits] [flang] fd0ffee - [flang] fix unused variable (NFC) (#227768)

[via flang-commits]

Author: Kelvin Li
Date: 2026-10-01T18:06:37-04:00
New Revision: fd0ffeea338eed99f799ce450cb744a9fab2b445

URL: https://github.com/llvm/llvm-project/commit/fd0ffeea338eed99f799ce450cb744a9fab2b445
DIFF: https://github.com/llvm/llvm-project/commit/fd0ffeea338eed99f799ce450cb744a9fab2b445.diff

LOG: [flang] fix unused variable (NFC) (#227768)

Added: 
    

Modified: 
    flang/lib/Lower/ConvertCall.cpp
    flang/lib/Lower/PFTBuilder.cpp
    flang/lib/Optimizer/OpenMP/MapInfoFinalization.cpp

Removed: 
    


################################################################################
diff  --git a/flang/lib/Lower/ConvertCall.cpp b/flang/lib/Lower/ConvertCall.cpp
index f2ce4b20803dd..6cb12b8208203 100644
--- a/flang/lib/Lower/ConvertCall.cpp
+++ b/flang/lib/Lower/ConvertCall.cpp
@@ -1462,8 +1462,8 @@ static PreparedDummyArgument preparePresentUserCallActualArgument(
   // Helpers to generate hlfir.copy_in operation and register the related
   // hlfir.copy_out creation.
   auto genCopyIn = [&](hlfir::Entity var, bool doCopyOut) -> hlfir::Entity {
-    auto baseBoxTy = mlir::dyn_cast<fir::BaseBoxType>(var.getType());
-    assert(baseBoxTy && "expect non simply contiguous variables to be boxes");
+    assert(mlir::dyn_cast<fir::BaseBoxType>(var.getType()) &&
+           "expect non simply contiguous variables to be boxes");
     mlir::Value tempBox = builder.createTemporary(loc, var.getType());
     auto copyIn = hlfir::CopyInOp::create(builder, loc, var, tempBox,
                                           /*var_is_present=*/mlir::Value{});

diff  --git a/flang/lib/Lower/PFTBuilder.cpp b/flang/lib/Lower/PFTBuilder.cpp
index 2bbf6eebc2c73..536ba708b1916 100644
--- a/flang/lib/Lower/PFTBuilder.cpp
+++ b/flang/lib/Lower/PFTBuilder.cpp
@@ -1117,8 +1117,8 @@ class PFTBuilder {
           },
           [&](const parser::AssignStmt &s) { // legacy label assignment
             auto &label = std::get<parser::Label>(s.t);
-            const auto *sym = std::get<parser::Name>(s.t).symbol;
-            assert(sym && "missing AssignStmt symbol");
+            assert(std::get<parser::Name>(s.t).symbol &&
+                   "missing AssignStmt symbol");
             auto labelIter{labelEvaluationMap->find(label)};
             assert(labelIter != labelEvaluationMap->end() &&
                    "assigned label has no evaluation");

diff  --git a/flang/lib/Optimizer/OpenMP/MapInfoFinalization.cpp b/flang/lib/Optimizer/OpenMP/MapInfoFinalization.cpp
index dfbf014f95b43..388b692b3f741 100644
--- a/flang/lib/Optimizer/OpenMP/MapInfoFinalization.cpp
+++ b/flang/lib/Optimizer/OpenMP/MapInfoFinalization.cpp
@@ -1605,7 +1605,7 @@ class MapInfoFinalizationPass
     // Make sure that updateUseDeviceDescriptorArgs was launched earlier
     auto arg = getUseDeviceAddrBlockArg(mapOp, *targetDataOp.getOperation());
     bool isArgBoxType = mlir::isa<fir::BaseBoxType>(arg.getType());
-    bool useArgLoad =
+    [[maybe_unused]] bool useArgLoad =
         arg.hasOneUse() && mlir::isa<fir::LoadOp>(*arg.use_begin()->getOwner());
     assert((isArgBoxType || useArgLoad) &&
            "Expected either BaseBox item or Load operation");