[PATCH] D42997: [AMDGPU] Fixed wait count reuse

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 6 17:57:38 PST 2018


rampitec created this revision.
rampitec added a reviewer: msearles.
Herald added subscribers: t-tye, tpr, dstuttard, yaxunl, nhaehnle, wdng, kzhuravl, arsenm.

The code reusing existing wait counts is incorrect since it keeps
adding new operands to an old instruction instead of replacing
the immediate. It was also effectively switched off by the condition
that wait count is not an AMDGPU::S_WAITCNT.

Also switched to BuildMI instead of creating instructions directly.


https://reviews.llvm.org/D42997

Files:
  lib/Target/AMDGPU/SIInsertWaitcnts.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42997.133123.patch
Type: text/x-patch
Size: 4341 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180207/6700bff2/attachment.bin>


More information about the llvm-commits mailing list