Skip to content

Commit 9e406a9

Browse files
frederik-harsenm
andauthored
Apply suggestions from code review
Co-authored-by: Matt Arsenault <[email protected]>
1 parent 5c4cae5 commit 9e406a9

File tree

2 files changed

+8
-10
lines changed

2 files changed

+8
-10
lines changed

llvm/test/CodeGen/AMDGPU/sdwa-peephole-cndmask-vop3-wave32.mir

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -150,8 +150,6 @@ body: |
150150
---
151151
name: missing-carry-def
152152
tracksRegLiveness: true
153-
registers:
154-
- { id: 0, class: sreg_32_xm0_xexec }
155153
body: |
156154
bb.0:
157155
liveins: $vgpr0
@@ -165,7 +163,7 @@ body: |
165163
; CHECK-NEXT: $vgpr0 = COPY [[V_CNDMASK_B32_e64_]]
166164
; CHECK-NEXT: SI_RETURN implicit $vgpr0
167165
%1:vgpr_32 = IMPLICIT_DEF
168-
%2:vgpr_32 = V_LSHRREV_B32_e64 16, undef %1, implicit $exec
166+
%2:vgpr_32 = V_LSHRREV_B32_e64 16, undef %1:sreg_32_xm0_xexec, implicit $exec
169167
%3:vgpr_32 = V_CNDMASK_B32_e64 0, 0, 0, %2, undef %0, implicit $exec
170168
$vgpr0 = COPY %3
171169
SI_RETURN implicit $vgpr0

llvm/test/CodeGen/AMDGPU/sdwa-peephole-cndmask-vop3-wave64.mir

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,9 @@ body: |
5050
; CHECK-NEXT: SI_RETURN implicit $vgpr0
5151
%0:sreg_64_xexec = IMPLICIT_DEF
5252
%1:vgpr_32 = IMPLICIT_DEF
53-
%2:vgpr_32 = V_PK_MAX_F16 8, undef %1, 8, undef %1, 0, 0, 0, 0, 0, implicit $mode, implicit $exec
53+
%2:vgpr_32 = V_PK_MAX_F16 8, %1, 8, %1, 0, 0, 0, 0, 0, implicit $mode, implicit $exec
5454
%3:vgpr_32 = V_LSHRREV_B32_e64 16, undef %2, implicit $exec
55-
%4:vgpr_32 = V_CNDMASK_B32_e64 0, 0, 0, %3, killed undef %0, implicit $exec
55+
%4:vgpr_32 = V_CNDMASK_B32_e64 0, 0, 0, %3, killed %0, implicit $exec
5656
$vgpr0 = COPY %4
5757
SI_RETURN implicit $vgpr0
5858
...
@@ -82,7 +82,7 @@ body: |
8282
%1:vgpr_32 = IMPLICIT_DEF
8383
%2:vgpr_32 = V_PK_MAX_F16 8, undef %1, 8, undef %1, 0, 0, 0, 0, 0, implicit $mode, implicit $exec
8484
%3:vgpr_32 = V_LSHRREV_B32_e64 16, %2, implicit $exec
85-
%4:vgpr_32 = V_CNDMASK_B32_e64 0, 0, 0, %3, killed undef %0, implicit $exec
85+
%4:vgpr_32 = V_CNDMASK_B32_e64 0, 0, 0, %3, killed %0, implicit $exec
8686
$vgpr0 = COPY %4
8787
$vgpr1 = COPY %0
8888
SI_RETURN implicit $vgpr0
@@ -110,9 +110,9 @@ body: |
110110
; CHECK-NEXT: SI_RETURN implicit $vgpr0
111111
%0:vgpr_32 = IMPLICIT_DEF
112112
%2:vgpr_32 = IMPLICIT_DEF
113-
%3:sreg_64_xexec = V_CMP_EQ_U32_e64 undef %0, 1, implicit $exec
114-
%5:vgpr_32 = V_LSHRREV_B32_e64 16, undef %2, implicit $exec
115-
V_CMP_EQ_U32_e32 1, undef %2, implicit-def $vcc, implicit $exec
113+
%3:sreg_64_xexec = V_CMP_EQ_U32_e64 %0, 1, implicit $exec
114+
%5:vgpr_32 = V_LSHRREV_B32_e64 16, %2, implicit $exec
115+
V_CMP_EQ_U32_e32 1, %2, implicit-def $vcc, implicit $exec
116116
%6:vgpr_32 = V_CNDMASK_B32_e64 0, 0, 0, %5, %3, implicit $exec
117117
%7:vgpr_32 = V_CNDMASK_B32_e32 killed %5, killed %5, implicit $vcc, implicit $exec
118118
$vgpr0 = COPY %6
@@ -165,7 +165,7 @@ body: |
165165
; CHECK-NEXT: $vgpr0 = COPY [[V_CNDMASK_B32_e64_]]
166166
; CHECK-NEXT: SI_RETURN implicit $vgpr0
167167
%1:vgpr_32 = IMPLICIT_DEF
168-
%2:vgpr_32 = V_LSHRREV_B32_e64 16, undef %1, implicit $exec
168+
%2:vgpr_32 = V_LSHRREV_B32_e64 16, %1, implicit $exec
169169
%3:vgpr_32 = V_CNDMASK_B32_e64 0, 0, 0, %2, undef %0, implicit $exec
170170
$vgpr0 = COPY %3
171171
SI_RETURN implicit $vgpr0

0 commit comments

Comments
 (0)