[all-commits] [llvm/llvm-project] 9f8e7c: AMDGPU: Create pseudo to real mapping for flat/buf...

[Matt Arsenault via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9f8e7c3a01bc071447634eaa3e7b41ae072a87fe
      https://github.com/llvm/llvm-project/commit/9f8e7c3a01bc071447634eaa3e7b41ae072a87fe
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2024-06-18 (Tue, 18 Jun 2024)

  Changed paths:
    M llvm/lib/Target/AMDGPU/AMDGPU.td
    M llvm/lib/Target/AMDGPU/BUFInstructions.td
    M llvm/lib/Target/AMDGPU/FLATInstructions.td
    M llvm/test/CodeGen/AMDGPU/fp-atomic-to-s_denormmode.mir

  Log Message:
  -----------
  AMDGPU: Create pseudo to real mapping for flat/buffer atomic fmin/fmax (#95591)

The global/flat/buffer atomic fmin/fmax situation is a mess. These
instructions have been renamed 3 times. We currently have
separate pseudos defined for the same opcodes with the different names
(e.g. GLOBAL_ATOMIC_MIN_F64 from gfx90a and GLOBAL_ATOMIC_FMIN_X2 from gfx10).

Use the _FMIN versions as the canonical name for the f32 versions. Use the
_MIN_F64 style as the canonical name for the f64 case. This is because
gfx90a has the most sensible names, but does not have the f32 versions.t sho

Wire through the pseudo to use for the instruction properties vs. the assembly
name like in other cases. This will simplify handling of direct atomicrmw selection.

This will simplify directly selecting these from atomicrmw.



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