[PATCH] D67771: [AMDGPU] fixed underflow in getOccupancyWithNumVGPRs

Thu Sep 19 12:40:37 PDT 2019

rampitec added a comment.

In D67771#1675537 <https://reviews.llvm.org/D67771#1675537>, @arsenm wrote:

> In D67771#1675524 <https://reviews.llvm.org/D67771#1675524>, @rampitec wrote:
>
> > In D67771#1675523 <https://reviews.llvm.org/D67771#1675523>, @arsenm wrote:
> >
> > > Testcase?
> >
> >
> > Bug is trivial and testcase must be huge to hit this condition. You need to have thousands of spills for this.
>
>
> I don't see how the number of spills factors in, only the number of used VGPRs

It happens in the scheduler. These are still virtual registers and there are no spills yet.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67771/new/

https://reviews.llvm.org/D67771