[PATCH] D23785: AMDGPU: Remove unneeded implicit exec defs
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 26 18:13:51 PDT 2016
arsenm added inline comments.
================
Comment at: lib/Target/AMDGPU/SIInstructions.td:1815-1820
@@ -1816,8 +1814,8 @@
-def SI_IF: PseudoInstSI <
+def SI_IF: CFPseudoInstSI <
(outs SReg_64:$dst), (ins SReg_64:$vcc, brtarget:$target),
- [(set i64:$dst, (int_amdgcn_if i1:$vcc, bb:$target))]> {
+ [(set i64:$dst, (int_amdgcn_if i1:$vcc, bb:$target))], 1, 1> {
let Constraints = "";
let Size = 8;
}
----------------
tstellarAMD wrote:
> Doesn't this expand to something that read/writes exec?
Yes. It still does here, it just sets the bit to CFPseudoInstSI instead of using the let blocks
https://reviews.llvm.org/D23785
More information about the llvm-commits
mailing list