Skip to content

Commit 0e3bf60

Browse files
authored
Remove hack to handle annotations in scf to cf (#7900)
Now that we integrated llvm fix we can remove the forked pass
1 parent 6ca2dda commit 0e3bf60

File tree

6 files changed

+2
-248
lines changed

6 files changed

+2
-248
lines changed

include/triton/Dialect/Triton/Transforms/Passes.td

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -90,15 +90,4 @@ def TritonLoopAwareCSE : Pass<"triton-loop-aware-cse", "mlir::ModuleOp"> {
9090
}];
9191
}
9292

93-
def TritonSCFToCF : Pass</*cli-arg*/"triton-scf-to-cf", /*Op*/"mlir::ModuleOp"> {
94-
let summary = "MLIR's SCF To CF plus some extra attributes propagation.";
95-
let description = [{
96-
This pass uses MLIR's SCF To CF pass as base. Additionally, it propagates
97-
some extra attributes to the converted CFG.
98-
TODO: upstream the llvm loop attribute propagation and remove this pass.
99-
}];
100-
101-
let dependentDialects = [];
102-
}
103-
10493
#endif

lib/Dialect/Triton/Transforms/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ add_triton_library(TritonTransforms
1313
RewriteTensorDescriptorToPointer.cpp
1414
ArithTypeConversion.cpp
1515
FunctionTypeConversion.cpp
16-
SCFToCF.cpp
1716

1817
DEPENDS
1918
TritonTransformsIncGen

lib/Dialect/Triton/Transforms/SCFToCF.cpp

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

python/src/passes.cc

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,6 @@ void init_triton_passes_ttgpuir(py::module &&m) {
9696

9797
void init_triton_passes_convert(py::module &&m) {
9898
using namespace mlir;
99-
ADD_PASS_WRAPPER_0("add_triton_scf_to_cf", mlir::triton::createTritonSCFToCF);
10099
ADD_PASS_WRAPPER_0("add_scf_to_cf", createSCFToControlFlowPass);
101100
ADD_PASS_WRAPPER_0("add_cf_to_llvmir", createConvertControlFlowToLLVMPass);
102101
ADD_PASS_WRAPPER_0("add_index_to_llvmir", createConvertIndexToLLVMPass);

third_party/amd/backend/compiler.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -287,7 +287,7 @@ def make_llir(src, metadata, options):
287287
# LDS size is determined by provided arch name.
288288
custom_lds_size = 0
289289
amd.passes.ttgpuir.add_optimize_lds_usage(pm, options.arch, custom_lds_size)
290-
passes.convert.add_triton_scf_to_cf(pm)
290+
passes.convert.add_scf_to_cf(pm)
291291
passes.convert.add_index_to_llvmir(pm)
292292

293293
amd.passes.ttgpuir.add_allocate_shared_memory(pm)

third_party/nvidia/backend/compiler.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -348,7 +348,7 @@ def make_llir(self, src, metadata, options, capability):
348348

349349
passes.ttgpuir.add_combine_tensor_select_and_if(pm)
350350
passes.ttgpuir.add_allocate_warp_groups(pm)
351-
passes.convert.add_triton_scf_to_cf(pm)
351+
passes.convert.add_scf_to_cf(pm)
352352
nvidia.passes.ttgpuir.add_allocate_shared_memory_nv(pm, capability, ptx_version)
353353
nvidia.passes.ttnvgpuir.add_allocate_tensor_memory(pm)
354354
if knobs.compilation.enable_experimental_consan:

0 commit comments

Comments
 (0)