[NFC][flang] Post changes for D127137 and D130027

[peixin]

Summary

D130027 adds one unused header in Bridge.cpp. D127137 needs one FIXME
as commented.

[arnamoy10]

LGTM

[clementval]

I think you can send these changes without review as NFC. Like that you can make two small commits with meaningful commit message. Here the commit title is not really useful.

[peixin]

In D130625#3682107, @clementval wrote:

I think you can send these changes without review as NFC. Like that you can make two small commits with meaningful commit message. Here the commit title is not really useful.

OK. I will split it into two small commits with meaningful title and commit message.

flang/lib/Lower/Bridge.cpp

 #include "flang/Optimizer/Support/FIRContext.h"
 #include "flang/Optimizer/Support/FatalError.h"
 #include "flang/Optimizer/Support/InternalNames.h"
 #include "flang/Optimizer/Transforms/Passes.h"
 #include "flang/Parser/parse-tree.h"
 #include "flang/Runtime/iostat.h"
 #include "flang/Semantics/tools.h"
 #include "mlir/Dialect/ControlFlow/IR/ControlFlowOps.h"
-#include "mlir/Dialect/OpenMP/OpenMPDialect.h"
 #include "mlir/IR/PatternMatch.h"
 #include "mlir/Parser/Parser.h"
 #include "mlir/Transforms/RegionUtils.h"
 #include "llvm/Support/CommandLine.h"
 #include "llvm/Support/Debug.h"
 #include "llvm/Support/ErrorHandling.h"
 
 #define DEBUG_TYPE "flang-lower-bridge"
fir::ExtendedValue exv = hexv.match(
         });
 
     // Replace all uses of the original with the clone/copy,
     // esepcially for loop bounds (that uses the variable being privatised)
     // since loop bounds use old values that need to be fixed by using the
     // new copied value.
     // Not able to use replaceAllUsesWith() because uses outside
     // the loop body should not use the clone.
+    // FIXME: Call privatization before the loop operation.
     mlir::Region &curRegion = getFirOpBuilder().getRegion();
     mlir::Value oldVal = fir::getBase(hexv);
     mlir::Value cloneVal = fir::getBase(exv);
     for (auto &oper : curRegion.getOps()) {
       for (unsigned int ii = 0; ii < oper.getNumOperands(); ++ii) {
         if (oper.getOperand(ii) == oldVal) {
           oper.setOperand(ii, cloneVal);
         }