Emit stackmaps for all indirect calls.

Since the JIT compiler can inline indirect calls, we need to emit
stackmaps for them too. Since we can't tell if an indirect call is an
intrisnic (which doesn't need stackmaps) this will inevitably create
some unneeded stackmap calls. But since they'll never be used, all they
cost is space in the binary.

Author: ptersilie

llvm/lib/Transforms/Yk/StackMaps.cpp

@@ -56,9 +56,10 @@ class YkStackmaps : public ModulePass {
             CallInst &CI = cast<CallInst>(I);
             if (CI.isInlineAsm())
               continue;
-            if (CI.isIndirectCall())
-              continue;
-            if (CI.getCalledFunction()->isIntrinsic())
+            // We don't need to insert stackmaps after intrinsics. But since we
+            // can't tell if an indirect call is an intrinsic at compile time,
+            // emit a stackmap in those cases too.
+            if (!CI.isIndirectCall() && CI.getCalledFunction()->isIntrinsic())
               continue;
             SMCalls.insert({&I, LA.getLiveVarsBefore(&I)});
           } else if ((isa<BranchInst>(I) &&