[PATCH] D54228: AMDGPU/InsertWaitcnts: Simplify pending events tracking

Tony Tye via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 19 08:27:25 PST 2018


t-tye added a comment.

In https://reviews.llvm.org/D54228#1302715, @nhaehnle wrote:

> Ping?
>
> I think the remarks by @t-tye point to a potentially useful optimization, but that should not be part of this patch.


Agree that optimization could be done as a separate patch. Would a Fixme comment be a good idea to record the idea in the appropriate place?


Repository:
  rL LLVM

https://reviews.llvm.org/D54228





More information about the llvm-commits mailing list