[PATCH] D17039: AMDGPU: Set flat_scratch from flat_scratch_init reg

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 11 22:36:06 PST 2016


arsenm closed this revision.
arsenm added a comment.

r260658


================
Comment at: lib/Target/AMDGPU/SIFrameLowering.cpp:78
@@ -86,1 +77,3 @@
 
+  if (MFI->hasFlatScratchInit() /*&& MFI->hasFlatInstructions()*/) {
+    // We don't need this if we only have spills since there is no user facing
----------------
tstellarAMD wrote:
> I'm not sure if this comment was intentional, but I think this should be dropped.
The condition should be there, but right now it doesn't do anything because flat instructions are detected in SILowerControlFlow


http://reviews.llvm.org/D17039





More information about the llvm-commits mailing list