From 236082a0fe4dcd0362c5c74923a93e3db6a01291 Mon Sep 17 00:00:00 2001 From: Richard-Qin-X Date: Thu, 13 Aug 2026 05:45:37 +0800 Subject: [CodeGen] Align pass manager ownership --- backend/unittests/CodeGen/LiveVariablesTest.cpp | 26 ++++++++++++++----------- 1 file changed, 15 insertions(+), 11 deletions(-) (limited to 'backend/unittests/CodeGen/LiveVariablesTest.cpp') diff --git a/backend/unittests/CodeGen/LiveVariablesTest.cpp b/backend/unittests/CodeGen/LiveVariablesTest.cpp index 577ecde..1cc4e91 100644 --- a/backend/unittests/CodeGen/LiveVariablesTest.cpp +++ b/backend/unittests/CodeGen/LiveVariablesTest.cpp @@ -36,9 +36,9 @@ void testLivenessRebuildAndVerifier(TestContext &Context, const rcc::TargetDescr Context.expect(StaleLiveIn.has_value(), "stale live-in setup must succeed"); setRebuildProperties(Function); - rcc::MachinePassManager Pipeline; + rcc::MachineFunctionPassManager Pipeline; auto Added = Pipeline.addPass(std::make_unique()); - rcc::MachineAnalysisManager Analyses; + rcc::MachineFunctionAnalysisManager Analyses; auto Run = Added ? Pipeline.run(Function, Target, Analyses) : rcc::Expected{std::unexpected(Added.error())}; if (!Run) @@ -169,8 +169,9 @@ class DummyAnalysis final { public: struct Result final {}; - [[nodiscard]] static rcc::Expected - run(rcc::MachineFunction &, const rcc::TargetDescription &, rcc::MachineAnalysisManager &) { + [[nodiscard]] static rcc::Expected run(rcc::MachineFunction &, + const rcc::TargetDescription &, + rcc::MachineFunctionAnalysisManager &) { return Result{}; } }; @@ -192,7 +193,7 @@ void testAnalysisRegistrationAndInvalidation(TestContext &Context, return; } - rcc::MachineAnalysisManager Analyses; + rcc::MachineFunctionAnalysisManager Analyses; auto FirstRegistration = rcc::registerStage11MachineAnalyses(Analyses); auto SecondRegistration = rcc::registerStage11MachineAnalyses(Analyses); auto Intervals = FirstRegistration @@ -211,7 +212,7 @@ void testAnalysisRegistrationAndInvalidation(TestContext &Context, Context.expect( Analyses.cachedResultCount(*Fixture->Function) == 1 && Analyses.hasCached(*Fixture->Function, - rcc::MachineAnalysisManager::id()), + rcc::MachineFunctionAnalysisManager::id()), "operand changes must retain SlotIndexes while invalidating liveness dependents"); auto Recomputed = Analyses.getResult(*Fixture->Function, Target); Context.expect(Recomputed && Analyses.cachedResultCount(*Fixture->Function) == 3, @@ -221,18 +222,21 @@ void testAnalysisRegistrationAndInvalidation(TestContext &Context, Context.expect( Analyses.cachedResultCount(*Fixture->Function) == 1 && Analyses.hasCached(*Fixture->Function, - rcc::MachineAnalysisManager::id()), + rcc::MachineFunctionAnalysisManager::id()), "layout changes must invalidate SlotIndexes and LiveIntervals only"); - rcc::MachineAnalysisManager Atomic; + rcc::MachineFunctionAnalysisManager Atomic; rcc::MachineAnalysisRegistration DummyRegistration; DummyRegistration.Name = "SlotIndexes"; auto Dummy = Atomic.registerAnalysis(std::move(DummyRegistration)); auto Rejected = rcc::registerStage11MachineAnalyses(Atomic); Context.expect( - Dummy && !Rejected && Atomic.isRegistered(rcc::MachineAnalysisManager::id()) && - !Atomic.isRegistered(rcc::MachineAnalysisManager::id()) && - !Atomic.isRegistered(rcc::MachineAnalysisManager::id()), + Dummy && !Rejected && + Atomic.isRegistered(rcc::MachineFunctionAnalysisManager::id()) && + !Atomic.isRegistered( + rcc::MachineFunctionAnalysisManager::id()) && + !Atomic.isRegistered( + rcc::MachineFunctionAnalysisManager::id()), "failed standard-analysis registration must leave the manager unchanged"); } -- cgit v1.2.3