[PATCH] D26730: AMDGPU/GlobalISel: Add support for simple shaders

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUCallLowering.cpp:39
                                         const Value *Val, unsigned VReg) const {
+  MIRBuilder.buildInstr(AMDGPU::S_ENDPGM);
   return true;
----------------
ab wrote:
> Is the returned vreg ignored because all functions are void?
Yes. For now only some graphics shaders are allowed to have return values which aren't necessary for the initial implementation


https://reviews.llvm.org/D26730