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

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 19 12:38:29 PDT 2019


arsenm added a comment.

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


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

https://reviews.llvm.org/D67771





More information about the llvm-commits mailing list