[PATCH] D152458: AMDGPU: Move AMDGPUAttributor run earlier

Pierre van Houtryve via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 9 00:40:14 PDT 2023


Pierre-vh accepted this revision as: Pierre-vh.
Pierre-vh added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/test/CodeGen/AMDGPU/simple-indirect-call.ll:46-48
+; GFX9-NEXT:    s_add_u32 flat_scratch_lo, s12, s17
+; GFX9-NEXT:    s_addc_u32 flat_scratch_hi, s13, 0
+; GFX9-NEXT:    s_add_u32 s0, s0, s17
----------------
nit: Do you know why there's more registers being used?
We go up to s31 for the call at the end so I guess the SGPR usage is the same overall, I'm just wondering.


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

https://reviews.llvm.org/D152458



More information about the llvm-commits mailing list