[llvm] Reapply "[MemProf] Reduce cloning overhead by sharing nodes when possible" (#102932) with fixes (PR #106623)

[Snehasish Kumar via llvm-commits]

================
@@ -2146,6 +2204,21 @@ bool ModuleCallsiteContextGraph::calleeMatchesFunc(
   return true;
 }
 
+bool ModuleCallsiteContextGraph::sameCallee(Instruction *Call1,
+                                            Instruction *Call2) {
+  auto *CB1 = dyn_cast<CallBase>(Call1);
----------------
snehasish wrote:

Do we need to check the dyn_cast returned values for nulls? If not then should they be dyn_cast in the first place?

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