[PATCH] D108022: [AMDGPU] Skip pseudo MIs in hazard recognizer
Stanislav Mekhanoshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 13 09:48:32 PDT 2021
rampitec added a comment.
In D108022#2943841 <https://reviews.llvm.org/D108022#2943841>, @arsenm wrote:
> In D108022#2943806 <https://reviews.llvm.org/D108022#2943806>, @cdevadas wrote:
>
>> In D108022#2943730 <https://reviews.llvm.org/D108022#2943730>, @arsenm wrote:
>>
>>> I think you're missing the check in AdvanceCycle
>>
>> There was a call to getNumWaitStates in AdvanceCycle. I presumed a zero waitstate here is equivalent to an early return.
>> Isn't it?
>> https://github.com/llvm/llvm-project/blob/main/llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp#L365
>
> I think the early return on isMetaInstruction avoids pushing the instruction into EmittedInstrs
I think AdvanceCycle shall do the same as with isMetaInstruction() if getNumWaitStates() returned 0.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108022/new/
https://reviews.llvm.org/D108022
More information about the llvm-commits
mailing list