[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