[PATCH] D84081: AMDGPU: Fix not accounting for constantexpr uses of LDS globals

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 20 08:31:45 PDT 2020


arsenm marked an inline comment as done.
arsenm added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/promote-alloca-to-lds-constantexpr-use.ll:8
+
+; This function cannot promote to using LDS because of the size of the
+; constant expression use in the function, which was previously not
----------------
hliao wrote:
> We may add 'FIXME' as well to keep these tests tracked as well.
No need, D84168 fixes this


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

https://reviews.llvm.org/D84081





More information about the llvm-commits mailing list