[all-commits] [llvm/llvm-project] c325d4: new test

Matt Arsenault via All-commits all-commits at lists.llvm.org
Tue Feb 18 00:32:26 PST 2025


  Branch: refs/heads/users/arsenm/amdgpu/isLegalOperand-fix-multiple-literal-logic-gfx10
  Home:   https://github.com/llvm/llvm-project
  Commit: c325d4bb56d859e09d3574b8198907d995273a54
      https://github.com/llvm/llvm-project/commit/c325d4bb56d859e09d3574b8198907d995273a54
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-02-18 (Tue, 18 Feb 2025)

  Changed paths:
    A llvm/test/CodeGen/AMDGPU/fold-literal-multiple-gfx10.mir

  Log Message:
  -----------
  new test


  Commit: ff7f2d18a46b019ee49a06c47bee317a49e1acd9
      https://github.com/llvm/llvm-project/commit/ff7f2d18a46b019ee49a06c47bee317a49e1acd9
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-02-18 (Tue, 18 Feb 2025)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SIInstrInfo.cpp
    M llvm/test/CodeGen/AMDGPU/eliminate-frame-index-v-add-co-u32.mir
    M llvm/test/CodeGen/AMDGPU/fold-literal-multiple-gfx10.mir
    M llvm/test/CodeGen/AMDGPU/fold-vgpr-copy.mir

  Log Message:
  -----------
  AMDGPU: Correct legal literal operand logic for multiple uses

The same literal can be used multiple times in an instruction,
not just once. We were not tracking the used value to verify this,
so correct this.

This helps avoid regressions in a future patch.


Compare: https://github.com/llvm/llvm-project/compare/c325d4bb56d8%5E...ff7f2d18a46b

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