[PATCH] D33139: AMDGPU/SI: Move the local memory usage related checking after calling convention checking in PromoteAlloca

Changpeng Fang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 12 10:22:50 PDT 2017


cfang created this revision.
Herald added subscribers: t-tye, tpr, dstuttard, yaxunl, nhaehnle, wdng.

Promoting Alloca to Vector and Promoting Alloca to LDS are two independent handling of Alloca and should not affect each other.
As a result, we should not give up promoting to vector if there is not enough LDS. This patch factors out the local memory usage 
related checking out and replace it after the calling convention checking.


https://reviews.llvm.org/D33139

Files:
  lib/Target/AMDGPU/AMDGPUPromoteAlloca.cpp
  test/CodeGen/AMDGPU/vector-alloca.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D33139.98796.patch
Type: text/x-patch
Size: 11017 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170512/1ccc9f3e/attachment.bin>


More information about the llvm-commits mailing list