[PATCH] D80644: DAG: Fix expansion of DYNAMIC_STACKALLOC for StackGrowsUp targets

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 27 10:17:20 PDT 2020


arsenm added a comment.

Actually on second look, I can't use this implementation directly which I should have noticed before (and I'm also not sure how my test passes with this bug). This fix would still be needed for StackGrowsUp targets, but AMDGPU also needs to scale the increment amount by the wavesize which isn't done here


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

https://reviews.llvm.org/D80644





More information about the llvm-commits mailing list