[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