From 5c5a71428a93fb225f0057790f41697fe2a0df92 Mon Sep 17 00:00:00 2001 From: Richard-Qin-X Date: Wed, 19 Aug 2026 17:25:55 +0800 Subject: [CodeGen] Remove non-LLVM machine DCE placements --- backend/lib/CodeGen/MachineOptimizationPipeline.cpp | 6 ------ backend/unittests/Target/RISCVLateCodeGenTest.cpp | 4 ++-- 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/backend/lib/CodeGen/MachineOptimizationPipeline.cpp b/backend/lib/CodeGen/MachineOptimizationPipeline.cpp index f3ce86d..817b2ca 100644 --- a/backend/lib/CodeGen/MachineOptimizationPipeline.cpp +++ b/backend/lib/CodeGen/MachineOptimizationPipeline.cpp @@ -74,9 +74,6 @@ Expected addPreRegisterAllocationOptimizationPipeline( return std::unexpected(Added.error()); Added = addPass(Pipeline, TargetInfo.Combiner, MachineCombinerStage::PreRegisterAllocation); - if (!Added) - return std::unexpected(Added.error()); - Added = addPass(Pipeline); if (!Added) return std::unexpected(Added.error()); return addPass(Pipeline, TargetInfo.SchedulingModel); @@ -103,9 +100,6 @@ addPostRegisterAllocationPipeline(MachineFunctionPassManager &Pipeline, if (!Added) return std::unexpected(Added.error()); Added = addPass(Pipeline, TargetInfo.FrameLowering); - if (!Added) - return std::unexpected(Added.error()); - Added = addPass(Pipeline); if (!Added) return std::unexpected(Added.error()); Added = addPass(Pipeline, TargetInfo.PseudoExpansion); diff --git a/backend/unittests/Target/RISCVLateCodeGenTest.cpp b/backend/unittests/Target/RISCVLateCodeGenTest.cpp index 0deebc6..2b165c4 100644 --- a/backend/unittests/Target/RISCVLateCodeGenTest.cpp +++ b/backend/unittests/Target/RISCVLateCodeGenTest.cpp @@ -1138,7 +1138,7 @@ void testCanonicalOptimizationPipelines(TestContext &Context, SelectedPipeline.size() == 6, "canonical selected-SSA optimization pipeline must register completely"); Context.expect(rcc::addPreRegisterAllocationOptimizationPipeline(PreRAPipeline, TargetInfo) && - PreRAPipeline.size() == 4, + PreRAPipeline.size() == 3, "canonical pre-RA optimization pipeline must register completely"); auto Function = makeFunction(Context, "canonical_post_ra_pipeline", 21); @@ -1180,7 +1180,7 @@ void testCanonicalOptimizationPipelines(TestContext &Context, rcc::MachineFunctionPassManager Pipeline; auto Added = rcc::addPostRegisterAllocationPipeline(Pipeline, TargetInfo); - Context.expect(Added && Pipeline.size() == 20, + Context.expect(Added && Pipeline.size() == 19, "canonical post-RA pipeline must register every required pass"); if (!Added) return; -- cgit v1.2.3