Author: Teja Alaghari Date: 2026-01-13T10:18:55Z New Revision: c4bec64591ef907dc3b7109242fb0ce168f8e201
URL: https://github.com/llvm/llvm-project/commit/c4bec64591ef907dc3b7109242fb0ce168f8e201 DIFF: https://github.com/llvm/llvm-project/commit/c4bec64591ef907dc3b7109242fb0ce168f8e201.diff LOG: [TwoAddressInstruction] Track MadeChange when eliminating REG_SEQUENCE (#173535) When `eliminateRegSequence()` is called, the pass modifies the `MachineFunction` but `MadeChange` was not being set to true. This causes the pass to incorrectly return `PreservedAnalyses::all()` even though changes were made. Added: Modified: llvm/lib/CodeGen/TwoAddressInstructionPass.cpp Removed: ################################################################################ diff --git a/llvm/lib/CodeGen/TwoAddressInstructionPass.cpp b/llvm/lib/CodeGen/TwoAddressInstructionPass.cpp index 5d919b5a8000c..dbad06623754c 100644 --- a/llvm/lib/CodeGen/TwoAddressInstructionPass.cpp +++ b/llvm/lib/CodeGen/TwoAddressInstructionPass.cpp @@ -1865,8 +1865,10 @@ bool TwoAddressInstructionImpl::run() { // Expand REG_SEQUENCE instructions. This will position mi at the first // expanded instruction. - if (mi->isRegSequence()) + if (mi->isRegSequence()) { eliminateRegSequence(mi); + MadeChange = true; + } DistanceMap.insert(std::make_pair(&*mi, ++Dist)); _______________________________________________ llvm-branch-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
