[all-commits] [llvm/llvm-project] a99966: [AMDGPU] Scheduler: fix RP calculation for a MBB w...

Valery via All-commits all-commits at lists.llvm.org
Wed Mar 8 03:20:25 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a999669982d0cedacbb7371c96fce95682d582e1
      https://github.com/llvm/llvm-project/commit/a999669982d0cedacbb7371c96fce95682d582e1
  Author: Valery Pykhtin <valery.pykhtin at gmail.com>
  Date:   2023-03-08 (Wed, 08 Mar 2023)

  Changed paths:
    M llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp

  Log Message:
  -----------
  [AMDGPU] Scheduler: fix RP calculation for a MBB with one successor

We reuse live registers after tracking one MBB as live-ins to the successor MBB
if the successor is only one but we don't check if the successor has other predecessors.

`A   B`
` \ /`
`  C`

A and B have one successor but C has live-ins defined by A and B and therefore should be
initialized using LIS.

This fixes 83 lit tests out if 420 with EXPENSIVE_CHECK enabled.

Reviewed By: rampitec

Differential Revision: https://reviews.llvm.org/D136918




More information about the All-commits mailing list