[PATCH] D148119: [AMDGPU] Fix amdgpu_gfx tail-call test

[Sebastian Neubauer via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rGfee3980df5c6: [AMDGPU] Fix amdgpu_gfx tail-call test (authored by sebastian-ne).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D148119/new/

https://reviews.llvm.org/D148119

Files:
  llvm/test/CodeGen/AMDGPU/tail-call-amdgpu-gfx.ll


Index: llvm/test/CodeGen/AMDGPU/tail-call-amdgpu-gfx.ll
===================================================================
--- llvm/test/CodeGen/AMDGPU/tail-call-amdgpu-gfx.ll
+++ llvm/test/CodeGen/AMDGPU/tail-call-amdgpu-gfx.ll
@@ -2,14 +2,17 @@
 ; RUN: llc -mtriple=amdgcn--amdpal -mcpu=gfx900 -verify-machineinstrs < %s | FileCheck -check-prefixes=GCN -enable-var-scope %s
 ; RUN: llc -global-isel -mtriple=amdgcn--amdpal -mcpu=gfx900 -verify-machineinstrs < %s | FileCheck -check-prefixes=GCN -enable-var-scope %s
 
-; Callee with SGPR and VGPR arguments
-define hidden amdgpu_gfx float @callee(float %v.arg0, float inreg %s.arg1) {
+; FIXME: @caller uses s[4:5] to store the address of @callee.
+;        These registers are callee-save in the amdgpu_gfx calling convention, so they must not be clobbered, but they are clobbered here.
+
+; Callee with VGPR arguments
+define hidden amdgpu_gfx float @callee(float %v.arg0, float %v.arg1) {
 ; GCN-LABEL: callee:
 ; GCN:       ; %bb.0:
 ; GCN-NEXT:    s_waitcnt vmcnt(0) expcnt(0) lgkmcnt(0)
-; GCN-NEXT:    v_add_f32_e32 v0, s4, v0
+; GCN-NEXT:    v_add_f32_e32 v0, v0, v1
 ; GCN-NEXT:    s_setpc_b64 s[30:31]
-  %add = fadd float %v.arg0, %s.arg1
+  %add = fadd float %v.arg0, %v.arg1
   ret float %add
 }
 
@@ -17,32 +20,23 @@
 ; GCN-LABEL: caller:
 ; GCN:       ; %bb.0:
 ; GCN-NEXT:    s_waitcnt vmcnt(0) expcnt(0) lgkmcnt(0)
-; GCN-NEXT:    s_mov_b32 s36, s33
-; GCN-NEXT:    s_mov_b32 s33, s32
 ; GCN-NEXT:    s_xor_saveexec_b64 s[34:35], -1
-; GCN-NEXT:    buffer_store_dword v1, off, s[0:3], s33 ; 4-byte Folded Spill
+; GCN-NEXT:    buffer_store_dword v2, off, s[0:3], s32 ; 4-byte Folded Spill
 ; GCN-NEXT:    s_mov_b64 exec, s[34:35]
-; GCN-NEXT:    v_writelane_b32 v1, s4, 0
-; GCN-NEXT:    s_addk_i32 s32, 0x400
-; GCN-NEXT:    v_writelane_b32 v1, s30, 1
+; GCN-NEXT:    v_writelane_b32 v2, s4, 0
+; GCN-NEXT:    v_writelane_b32 v2, s5, 1
+; GCN-NEXT:    s_getpc_b64 s[4:5]
+; GCN-NEXT:    s_add_u32 s4, s4, callee at rel32@lo+4
+; GCN-NEXT:    s_addc_u32 s5, s5, callee at rel32@hi+12
 ; GCN-NEXT:    v_add_f32_e32 v0, 1.0, v0
-; GCN-NEXT:    s_mov_b32 s4, 2.0
-; GCN-NEXT:    v_writelane_b32 v1, s31, 2
-; GCN-NEXT:    s_getpc_b64 s[34:35]
-; GCN-NEXT:    s_add_u32 s34, s34, callee at rel32@lo+4
-; GCN-NEXT:    s_addc_u32 s35, s35, callee at rel32@hi+12
-; GCN-NEXT:    s_swappc_b64 s[30:31], s[34:35]
-; GCN-NEXT:    v_readlane_b32 s31, v1, 2
-; GCN-NEXT:    v_readlane_b32 s30, v1, 1
-; GCN-NEXT:    v_readlane_b32 s4, v1, 0
+; GCN-NEXT:    v_mov_b32_e32 v1, 2.0
+; GCN-NEXT:    v_readlane_b32 s5, v2, 1
+; GCN-NEXT:    v_readlane_b32 s4, v2, 0
 ; GCN-NEXT:    s_xor_saveexec_b64 s[34:35], -1
-; GCN-NEXT:    buffer_load_dword v1, off, s[0:3], s33 ; 4-byte Folded Reload
+; GCN-NEXT:    buffer_load_dword v2, off, s[0:3], s32 ; 4-byte Folded Reload
 ; GCN-NEXT:    s_mov_b64 exec, s[34:35]
-; GCN-NEXT:    s_addk_i32 s32, 0xfc00
-; GCN-NEXT:    s_mov_b32 s33, s36
-; GCN-NEXT:    s_waitcnt vmcnt(0)
-; GCN-NEXT:    s_setpc_b64 s[30:31]
+; GCN-NEXT:    s_setpc_b64 s[4:5]
   %add = fadd float %arg0, 1.0
-  %call = tail call amdgpu_gfx float @callee(float %add, float inreg 2.0)
+  %call = tail call amdgpu_gfx float @callee(float %add, float 2.0)
   ret float %call
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D148119.512825.patch
Type: text/x-patch
Size: 3231 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230412/5e126ecd/attachment.bin>