[llvm] [AMDGPU] Change handling of unsupported non-compute shaders with HSA (PR #126798)

[Robert Imschweiler via llvm-commits]

================
@@ -2933,7 +2937,7 @@ SDValue SITargetLowering::LowerFormalArguments(
 
   for (unsigned i = 0, e = Ins.size(), ArgIdx = 0; i != e; ++i) {
     const ISD::InputArg &Arg = Ins[i];
-    if (Arg.isOrigArg() && Skipped[Arg.getOrigArgIndex()]) {
+    if ((Arg.isOrigArg() && Skipped[Arg.getOrigArgIndex()]) || IsUnsupportedHsa) {
----------------
ro-i wrote:

> Oh right, the other special input setup stuff still needs to happen. I suppose you can rename the variable

Already done :)

> Although in that case we should stop requiring the CC lowering to set up the scratch resource. It's really a fixed constant

Hm, the function seems to do a lot of things (which I don't fully understand yet) in the case of `IsGraphics` and `IsEntryFunc`. Is it possible to pull all this into the error handling without too much code duplication?

https://github.com/llvm/llvm-project/pull/126798