[PATCH] D31124: AMDGPU/SI: Add lane tracking to SI Scheduler
Axel Davy via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 28 23:17:31 PDT 2017
axeldavy added a comment.
In https://reviews.llvm.org/D31124#712850, @vpykhtin wrote:
> Axel, thanks for the update.
>
> If you aren't going to add test for this, can you at least run your change on the debug build (or release with asserts) with sisched turned on by default over all AMDGPU lit tests? They mostly would fail on miscompare, but they should run without asserts.
I think it is a good idea to add a test, I think it'd have to be something with a lot of loads and involving a lot of subregs. I just hadn't time yet to write it.
Repository:
rL LLVM
https://reviews.llvm.org/D31124
More information about the llvm-commits
mailing list