[llvm] [MemProf] Don't mutate the function type when calling clone (PR #147829)

[Teresa Johnson via llvm-commits]

================
@@ -95,6 +95,7 @@ STATISTIC(NewMergedNodes, "Number of new nodes created during merging");
 STATISTIC(NonNewMergedNodes, "Number of non new nodes used during merging");
 STATISTIC(MissingAllocForContextId,
           "Number of missing alloc nodes for context ids");
+STATISTIC(SkippedCallsCloning, "Number of calls skipped during cloning");
----------------
teresajohnson wrote:

I started out making it longer and more specific, but figured we'd need to look up the reason in the code if we ever encounter this in the wild anyway (it's a developer-focused stat). Any suggestions for something more specific but not too terse? The other place we might skip is if it required ICP and we couldn't legally perform that. How about "Number of calls skipped during cloning due to unexpected operand" ?

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