DAG: Set hasCalls on frame info earlier

[arsenm]

Summary

AMDGPU needs to know if calls are present when lowering arguments,
but this was only set later. Set it along with other stack properties.

[arsenm]

Skip intrinsics

[arsenm]

This seems to break a variety of tests in other targets. Maybe a smarter version of isLoweredToCall is needed?

[arsenm]

Make it a target hook.

There are some special cases that make it difficult to make this
test generically. SystemZ emits some libcalls as special instructions,
and this depends on whether the target will end up emitting a
tail call.

[arsenm]

ping

[arsenm]

I found a better solution which avoids needing to predict this from a call

lib/CodeGen/SelectionDAG/FunctionLoweringInfo.cpp

for (const Instruction &I : BB) {
               TLI->ComputeConstraintToUse(Op, SDValue(), DAG);
               std::pair<unsigned, const TargetRegisterClass *> PhysReg =
                   TLI->getRegForInlineAsmConstraint(TRI, Op.ConstraintCode,
                                                     Op.ConstraintVT);
               if (PhysReg.first == SP)
                 MF->getFrameInfo().setHasOpaqueSPAdjustment(true);
             }
           }
+        } else if (const Function *F = CS.getCalledFunction()) {
+          if (!F->isIntrinsic())
+            MF->getFrameInfo().setHasCalls(true);
         }
       }
 
       // Look for calls to the @llvm.va_start intrinsic. We can omit some
       // prologue boilerplate for variadic functions that don't examine their
       // arguments.
       if (const auto *II = dyn_cast<IntrinsicInst>(&I)) {
         if (II->getIntrinsicID() == Intrinsic::vastart)