[PATCH] D136069: [AMDGPU] Scheduler: Don't revert the schedule if the register pressure isn't changed for a region

Valery Pykhtin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 18 04:47:44 PDT 2022


vpykhtin updated this revision to Diff 468482.
vpykhtin marked 3 inline comments as done.
vpykhtin added a comment.

Rebased, moved the check into shouldRevertScheduling of two stages:

OccInitialScheduleStage
ClusteredLowOccStage

I don't add it to the unclustered stage as this schedule should only be kept if some improvement in RP is achieved.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136069/new/

https://reviews.llvm.org/D136069

Files:
  llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp
  llvm/test/CodeGen/AMDGPU/load-global-i16.ll
  llvm/test/CodeGen/AMDGPU/machine-scheduler-sink-trivial-remats.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D136069.468482.patch
Type: text/x-patch
Size: 41222 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221018/8d7ce664/attachment-0001.bin>


More information about the llvm-commits mailing list