Skip to content

Commit 77a3538

Browse files
committed
[OpenMP][MLIR]OMPEarlyOutliningPass removal
This patch removes the OMPEarlyOutliningPass as it is no longer required. The implicit map operand capture has now been moved to the PFT lowering stage. Depends on #67318.
1 parent edea974 commit 77a3538

File tree

9 files changed

+3
-417
lines changed

9 files changed

+3
-417
lines changed

flang/include/flang/Optimizer/Transforms/Passes.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,8 +75,6 @@ std::unique_ptr<mlir::Pass>
7575
createAlgebraicSimplificationPass(const mlir::GreedyRewriteConfig &config);
7676
std::unique_ptr<mlir::Pass> createPolymorphicOpConversionPass();
7777

78-
std::unique_ptr<mlir::OperationPass<mlir::ModuleOp>>
79-
createOMPEarlyOutliningPass();
8078
std::unique_ptr<mlir::Pass> createOMPFunctionFilteringPass();
8179
std::unique_ptr<mlir::OperationPass<mlir::ModuleOp>>
8280
createOMPMarkDeclareTargetPass();

flang/include/flang/Optimizer/Transforms/Passes.td

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -318,18 +318,6 @@ def LoopVersioning : Pass<"loop-versioning", "mlir::func::FuncOp"> {
318318
let dependentDialects = [ "fir::FIROpsDialect" ];
319319
}
320320

321-
def OMPEarlyOutliningPass
322-
: Pass<"omp-early-target-outlining", "mlir::ModuleOp"> {
323-
let summary = "Outlines all target ops into separate functions";
324-
let description = [{
325-
This pass outlines all omp.target operations into individual functions.
326-
It is invoked in the front end after the initial FIR has been constructed.
327-
This pass is only needed when compiling for the target device to prevent
328-
the optimizer to perform transforms across target region boundaries.
329-
}];
330-
let constructor = "::fir::createOMPEarlyOutliningPass()";
331-
}
332-
333321
def OMPMarkDeclareTargetPass
334322
: Pass<"omp-mark-declare-target", "mlir::ModuleOp"> {
335323
let summary = "Marks all functions called by an OpenMP declare target function as declare target";

flang/include/flang/Tools/CLOptions.inc

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -274,10 +274,8 @@ inline void createHLFIRToFIRPassPipeline(
274274
inline void createOpenMPFIRPassPipeline(
275275
mlir::PassManager &pm, bool isTargetDevice) {
276276
pm.addPass(fir::createOMPMarkDeclareTargetPass());
277-
if (isTargetDevice) {
278-
pm.addPass(fir::createOMPEarlyOutliningPass());
277+
if (isTargetDevice)
279278
pm.addPass(fir::createOMPFunctionFilteringPass());
280-
}
281279
}
282280

283281
#if !defined(FLANG_EXCLUDE_CODEGEN)

flang/lib/Optimizer/Transforms/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ add_flang_library(FIRTransforms
1717
AddDebugFoundation.cpp
1818
PolymorphicOpConversion.cpp
1919
LoopVersioning.cpp
20-
OMPEarlyOutlining.cpp
2120
OMPFunctionFiltering.cpp
2221
OMPMarkDeclareTarget.cpp
2322
VScaleAttr.cpp

flang/lib/Optimizer/Transforms/OMPEarlyOutlining.cpp

Lines changed: 0 additions & 303 deletions
This file was deleted.

0 commit comments

Comments
 (0)