[PATCH] D21941: AMDGPU: Set isConvergent on v_cmpx* instructions

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 5 10:19:41 PDT 2016

arsenm added inline comments.

Comment at: lib/Target/AMDGPU/SIInstrInfo.td:2294
@@ -2293,1 +2293,3 @@
     let SchedRW = sched;
+    let hasSideEffects = DefExec;
+    let isConvergent = DefExec;
tstellarAMD wrote:
> Why do we need to set hasSideEffects here?
It seems like writing to exec should count as a side effect, although currently the realgar bit instruction manipulations will not have this set


More information about the llvm-commits mailing list