[llvm] [AMDGPU] SIInstrInfo: Fix resultDependsOnExec for VOPC instructions (PR #134629)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 17 10:24:35 PDT 2025
================
@@ -147,12 +147,16 @@ bool SIInstrInfo::isReallyTriviallyReMaterializable(
}
// Returns true if the scalar result of a VALU instruction depends on exec.
-static bool resultDependsOnExec(const MachineInstr &MI) {
+bool SIInstrInfo::resultDependsOnExec(const MachineInstr &MI) const {
// Ignore comparisons which are only used masked with exec.
// This allows some hoisting/sinking of VALU comparisons.
if (MI.isCompare()) {
const MachineRegisterInfo &MRI = MI.getParent()->getParent()->getRegInfo();
----------------
arsenm wrote:
Can sink this down (though actually I'd hope MRI would just be passed in to start with)
https://github.com/llvm/llvm-project/pull/134629
More information about the llvm-commits
mailing list