[PATCH] D89753: [HazardRec] Allow inserting multiple wait-states simultaneously

Austin Kerbow via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 20 11:52:21 PDT 2020


kerbowa added a comment.

In D89753#2342419 <https://reviews.llvm.org/D89753#2342419>, @rampitec wrote:

> Do we properly count number of pre-existing wait states if we have s_nop > 0?

In "HazardRecognizerMode", yes. It uses `SIInstrInfo::getNumWaitStates` which accounts for noops with immediates > 0.

The post-RA scheduler should not insert these combined noops so we did not handle it there.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89753



More information about the llvm-commits mailing list