From 0ffdf65eaa05209655601a1059232da6605162a3 Mon Sep 17 00:00:00 2001 From: Tom Stellard Date: Mon, 11 Apr 2016 20:38:40 +0000 Subject: Revert "AMDGPU/SI: Do not generate s_waitcnt after ds_permute/ds_bpermute" This reverts commit r263720. Just confirmed that s_waitcnt is required after ds_permute/ds_bpermute. llvm-svn: 265992 --- .../test/CodeGen/AMDGPU/llvm.amdgcn.ds.bpermute.ll | 25 +------------------ llvm/test/CodeGen/AMDGPU/llvm.amdgcn.ds.permute.ll | 29 +++------------------- 2 files changed, 4 insertions(+), 50 deletions(-) (limited to 'llvm/test') diff --git a/llvm/test/CodeGen/AMDGPU/llvm.amdgcn.ds.bpermute.ll b/llvm/test/CodeGen/AMDGPU/llvm.amdgcn.ds.bpermute.ll index d9c87b13604..fa12ee2fe92 100644 --- a/llvm/test/CodeGen/AMDGPU/llvm.amdgcn.ds.bpermute.ll +++ b/llvm/test/CodeGen/AMDGPU/llvm.amdgcn.ds.bpermute.ll @@ -4,34 +4,11 @@ declare i32 @llvm.amdgcn.ds.bpermute(i32, i32) #0 ; FUNC-LABEL: {{^}}ds_bpermute: ; CHECK: ds_bpermute_b32 v{{[0-9]+}}, v{{[0-9]+}}, v{{[0-9]+}} +; CHECK: s_waitcnt lgkmcnt define void @ds_bpermute(i32 addrspace(1)* %out, i32 %index, i32 %src) nounwind { %bpermute = call i32 @llvm.amdgcn.ds.bpermute(i32 %index, i32 %src) #0 store i32 %bpermute, i32 addrspace(1)* %out, align 4 ret void } -; FUNC-LABEL: {{^}}bpermute_no_waitcnt_test: -; CHECK: s_cbranch_scc1 -; CHECK: ds_bpermute_b32 v{{[0-9]+}}, v{{[0-9]+}}, v{{[0-9]+}} -; CHECK-NOT: s_waitcnt -define void @bpermute_no_waitcnt_test(i32 addrspace(1)* %out, i32 %cond) { -entry: - - %tmp = icmp eq i32 %cond, 0 - br i1 %tmp, label %if, label %else - -if: ; preds = %entry - - %bpermute = call i32 @llvm.amdgcn.ds.bpermute(i32 0, i32 0) #0 - br label %endif - -else: ; preds = %entry - br label %endif - -endif: - %val = phi i32 [ %bpermute, %if ], [0, %else] ; preds = %else, %if - store i32 %val, i32 addrspace(1)* %out, align 4 - ret void -} - attributes #0 = { nounwind readnone convergent } diff --git a/llvm/test/CodeGen/AMDGPU/llvm.amdgcn.ds.permute.ll b/llvm/test/CodeGen/AMDGPU/llvm.amdgcn.ds.permute.ll index 2f774abd1c6..f0ee48ff448 100644 --- a/llvm/test/CodeGen/AMDGPU/llvm.amdgcn.ds.permute.ll +++ b/llvm/test/CodeGen/AMDGPU/llvm.amdgcn.ds.permute.ll @@ -4,34 +4,11 @@ declare i32 @llvm.amdgcn.ds.permute(i32, i32) #0 ; FUNC-LABEL: {{^}}ds_permute: ; CHECK: ds_permute_b32 v{{[0-9]+}}, v{{[0-9]+}}, v{{[0-9]+}} +; CHECK: s_waitcnt lgkmcnt define void @ds_permute(i32 addrspace(1)* %out, i32 %index, i32 %src) nounwind { - %permute = call i32 @llvm.amdgcn.ds.permute(i32 %index, i32 %src) #0 - store i32 %permute, i32 addrspace(1)* %out, align 4 + %bpermute = call i32 @llvm.amdgcn.ds.permute(i32 %index, i32 %src) #0 + store i32 %bpermute, i32 addrspace(1)* %out, align 4 ret void } -; FUNC-LABEL: {{^}}permute_no_waitcnt_test: -; CHECK: s_cbranch_scc1 -; CHECK: ds_permute_b32 v{{[0-9]+}}, v{{[0-9]+}}, v{{[0-9]+}} -; CHECK-NOT: s_waitcnt -define void @permute_no_waitcnt_test(i32 addrspace(1)* %out, i32 %cond) { -entry: - - %tmp = icmp eq i32 %cond, 0 - br i1 %tmp, label %if, label %else - -if: ; preds = %entry - %permute = call i32 @llvm.amdgcn.ds.permute(i32 0, i32 0) #0 - br label %endif - -else: ; preds = %entry - br label %endif - -endif: - %val = phi i32 [ %permute, %if ], [0, %else] ; preds = %else, %if - store i32 %val, i32 addrspace(1)* %out, align 4 - ret void -} - - attributes #0 = { nounwind readnone convergent } -- cgit v1.2.3