[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