[all-commits] [llvm/llvm-project] 2c753b: new test

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


  Branch: refs/heads/users/arsenm/amdgpu/isLegalOperand-fix-multiple-literal-logic-gfx10
  Home:   https://github.com/llvm/llvm-project
  Commit: 2c753b0d6d727d4ccd561bd14bd8077483878964
      https://github.com/llvm/llvm-project/commit/2c753b0d6d727d4ccd561bd14bd8077483878964
  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: 57468eb659ec2de38386e670b520835ddfafcdae
      https://github.com/llvm/llvm-project/commit/57468eb659ec2de38386e670b520835ddfafcdae
  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/ff7f2d18a46b...57468eb659ec

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