[llvm] [AMDGPU] SIInstrInfo: Fix resultDependsOnExec for VOPC instructions (PR #134629)

Frederik Harwath via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 7 07:29:26 PDT 2025


================
@@ -151,6 +151,9 @@ static bool resultDependsOnExec(const MachineInstr &MI) {
   // Ignore comparisons which are only used masked with exec.
   // This allows some hoisting/sinking of VALU comparisons.
   if (MI.isCompare()) {
+    if (SIInstrInfo::isVOPC(MI))
----------------
frederik-h wrote:

@jayfoad But the implicit write to VCC doesn't "depend on exec", right? Should we adjust the comment?

https://github.com/llvm/llvm-project/pull/134629


More information about the llvm-commits mailing list