[PATCH] D101430: [AMDGPU] Refactor hazard recognition IsHazardFn and IsExpiredFn

Carl Ritson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 28 01:08:46 PDT 2021


critson created this revision.
critson added reviewers: foad, rampitec.
Herald added subscribers: kerbowa, jfb, hiraditya, t-tye, tpr, dstuttard, yaxunl, nhaehnle, jvesely, kzhuravl, arsenm.
critson requested review of this revision.
Herald added subscribers: llvm-commits, wdng.
Herald added a project: LLVM.

While counting wait states remove early termination probe using
nullptr with IsExpiredFn. This is the only functional change.

Having removed the use of null pointers, refactor IsHazardFn and
IsExpiredFn to use constant references as these should not be
mutating the instructions visited.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D101430

Files:
  llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp
  llvm/lib/Target/AMDGPU/GCNHazardRecognizer.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D101430.341091.patch
Type: text/x-patch
Size: 30587 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210428/5debb04c/attachment.bin>


More information about the llvm-commits mailing list