[all-commits] [llvm/llvm-project] da591d: [GlobalISel][TableGen] Take first result for multi...

[Björn Pettersson via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: da591d390e7f865c846d12dc5559875eca347c28
      https://github.com/llvm/llvm-project/commit/da591d390e7f865c846d12dc5559875eca347c28
  Author: Bjorn Pettersson <bjorn.a.pettersson at ericsson.com>
  Date:   2024-03-02 (Sat, 02 Mar 2024)

  Changed paths:
    M llvm/test/CodeGen/AMDGPU/GlobalISel/inst-select-fabs.mir
    M llvm/test/CodeGen/AMDGPU/GlobalISel/inst-select-fneg.mir
    M llvm/test/CodeGen/AMDGPU/GlobalISel/inst-select-sext.mir
    M llvm/test/CodeGen/AMDGPU/GlobalISel/inst-select-zext.mir
    A llvm/test/TableGen/GlobalISelEmitter-multiple-output-reject.td
    M llvm/test/TableGen/GlobalISelEmitter-multiple-output.td
    M llvm/utils/TableGen/GlobalISelEmitter.cpp

  Log Message:
  -----------
  [GlobalISel][TableGen] Take first result for multi-output instructions (#81130)

Previously, tblgen would reject patterns where one of its nested
instructions produced more than one result. These arise when the
instruction definition contains 'outs' as well as 'Defs'. This patch
fixes that by always taking the first result, which is how these
situations are handled in SelectionIDAG.

Original patch: https://reviews.llvm.org/D86617
Continued as: https://github.com/llvm/llvm-project/pull/81130



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