[PATCH] D31124: AMDGPU/SI: Add lane tracking to SI Scheduler

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 22 15:08:43 PDT 2017


rampitec added a comment.

In https://reviews.llvm.org/D31124#707998, @axeldavy wrote:

> Use reference for the input argument of getPairsForRegs.


Thank you. It looks good, but w/o reference previous version should not been working. I.e. it looks like this code is not covered by any test.
I think you need to add tests for this.


Repository:
  rL LLVM

https://reviews.llvm.org/D31124





More information about the llvm-commits mailing list