[all-commits] [llvm/llvm-project] 288902: 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: 288902beb48d921149898b6a15d3097b8d2450e9
      https://github.com/llvm/llvm-project/commit/288902beb48d921149898b6a15d3097b8d2450e9
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-02-24 (Mon, 24 Feb 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: c7000809185ff3a209b8d8ccac7a0736701ead14
      https://github.com/llvm/llvm-project/commit/c7000809185ff3a209b8d8ccac7a0736701ead14
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-02-24 (Mon, 24 Feb 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.


Compare: https://github.com/llvm/llvm-project/compare/90a9d04b4f3c...c7000809185f

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