[all-commits] [llvm/llvm-project] b0db9c: [AMDGPU] Insert V_NOP after V_PERM_PK16 (gfx1250 h...

hidekisaito via All-commits all-commits at lists.llvm.org
Mon Sep 28 10:45:46 PDT 2026


  Branch: refs/heads/users/hidekisaito/perm-pk16-hazard
  Home:   https://github.com/llvm/llvm-project
  Commit: b0db9ca71d1ad5832b6f88632a72476671e00a21
      https://github.com/llvm/llvm-project/commit/b0db9ca71d1ad5832b6f88632a72476671e00a21
  Author: Hideki Saito <hidekido at amd.com>
  Date:   2026-09-28 (Mon, 28 Sep 2026)

  Changed paths:
    M llvm/lib/Target/AMDGPU/AMDGPU.td
    M llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp
    M llvm/lib/Target/AMDGPU/GCNHazardRecognizer.h
    M llvm/lib/Target/AMDGPU/SIInstrInfo.cpp
    M llvm/lib/Target/AMDGPU/SIInstrInfo.h
    M llvm/test/CodeGen/AMDGPU/llvm.amdgcn.perm.pk.ll
    A llvm/test/CodeGen/AMDGPU/remove-short-exec-branches-vperm-pk16.mir
    A llvm/test/CodeGen/AMDGPU/vperm-pk16-exec-hazard.ll
    A llvm/test/CodeGen/AMDGPU/vperm-pk16-exec-hazard.mir

  Log Message:
  -----------
  [AMDGPU] Insert V_NOP after V_PERM_PK16 (gfx1250 hazard)

On gfx1250 the V_PERM_PK16 family (V_PERM_PK16_B4/B6/B8_U4) has a hazard:
the instruction must be immediately followed by a "safe" instruction that
issues on the pipe which clears the hazard. Insert V_NOP as needed.

Also updated hasUnwantedEffectsWhenEXECEmpty() to ensure V_PERM_PK16 and
the inserted V_NOP are under non-zero EXEC.

Fixes: ROCM-26041

Assisted-by: Opus 4.8 Medium


  Commit: 51154d80c3e11471161da8dc3728bbab1fd66f6d
      https://github.com/llvm/llvm-project/commit/51154d80c3e11471161da8dc3728bbab1fd66f6d
  Author: Hideki Saito <hidekido at amd.com>
  Date:   2026-09-28 (Mon, 28 Sep 2026)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SIInstrInfo.cpp
    M llvm/lib/Target/AMDGPU/SIInstrInfo.h
    M llvm/test/CodeGen/AMDGPU/llvm.amdgcn.perm.pk.ll
    M llvm/test/CodeGen/AMDGPU/remove-short-exec-branches-vperm-pk16.mir
    M llvm/test/CodeGen/AMDGPU/vperm-pk16-exec-hazard.ll
    M llvm/test/CodeGen/AMDGPU/vperm-pk16-exec-hazard.mir

  Log Message:
  -----------
  [AMDGPU] Ensure v_perm_pk16 hazard is applied to both gfx1250 and gfx1251.


Compare: https://github.com/llvm/llvm-project/compare/9fa7fc2847aa...51154d80c3e1

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


More information about the All-commits mailing list