[PATCH] D24285: AMDGPU: Remove code I think is dead

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 9 15:07:14 PDT 2016


arsenm added a comment.

In https://reviews.llvm.org/D24285#537988, @nhaehnle wrote:

> I'm not deep enough into this code to really say what's expected, but I remember changing something there because the ADD path _was_ triggered (although I'm not sure whether it was here or in materializeFrameBaseRegister or both). The offending shader is attached to the bug report at https://bugs.freedesktop.org/show_bug.cgi?id=96602, perhaps you could check if that compiles with this change?


That compiles (as well as the testcase you committed with that fix)


https://reviews.llvm.org/D24285





More information about the llvm-commits mailing list