[all-commits] [llvm/llvm-project] 7cd0e6: AMDGPU: Add amdgpu-num-agpr attribute to control A...

[Matt Arsenault via All-commits]

  Branch: refs/heads/users/arsenm/amdgpu/add-amdgpu-num-agpr-attribute
  Home:   https://github.com/llvm/llvm-project
  Commit: 7cd0e6c7c087432600f0062c3cb3edd4c9b72f7a
      https://github.com/llvm/llvm-project/commit/7cd0e6c7c087432600f0062c3cb3edd4c9b72f7a
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-03-06 (Thu, 06 Mar 2025)

  Changed paths:
    M llvm/docs/AMDGPUUsage.rst
    M llvm/lib/Target/AMDGPU/SIRegisterInfo.cpp
    A llvm/test/CodeGen/AMDGPU/amdgpu-num-agpr.ll

  Log Message:
  -----------
  AMDGPU: Add amdgpu-num-agpr attribute to control AGPR allocation

This provides a range to decide how to subdivide the vector register
budget on gfx90a+. A single value declares the minimum AGPRs that
should be allocatable. Eventually this should replace amdgpu-no-agpr.

I want this primarily for testing agpr allocation behavior. We should
have a heuristic try to detect a reasonable number of AGPRs to keep
allocatable.


  Commit: 5d1b03ebb8b1f9b474c77cf84b9cbf81a3c55295
      https://github.com/llvm/llvm-project/commit/5d1b03ebb8b1f9b474c77cf84b9cbf81a3c55295
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-03-06 (Thu, 06 Mar 2025)

  Changed paths:
    A llvm/test/CodeGen/AMDGPU/amdgpu-no-agprs-violations.ll

  Log Message:
  -----------
  Add test for what happens when you violate the attribute

We hit the allocation error, but we probably should treat it as UB and only
emit a warning.


  Commit: 9d313bb55d871a4a8235358bbc559ae0d9a4ba1d
      https://github.com/llvm/llvm-project/commit/9d313bb55d871a4a8235358bbc559ae0d9a4ba1d
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-03-06 (Thu, 06 Mar 2025)

  Changed paths:
    M llvm/docs/AMDGPUUsage.rst
    M llvm/lib/Target/AMDGPU/SIRegisterInfo.cpp
    M llvm/test/CodeGen/AMDGPU/amdgpu-no-agprs-violations.ll
    M llvm/test/CodeGen/AMDGPU/amdgpu-num-agpr.ll

  Log Message:
  -----------
  Rename to amdgpu-agpr-alloc, but having a hard time finding a good name


Compare: https://github.com/llvm/llvm-project/compare/f2c130ce4ea8...9d313bb55d87

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications