[llvm] [MemProf] Handle missing tail call frames (PR #75823)

Snehasish Kumar via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 10 14:25:48 PST 2024


================
@@ -104,6 +114,12 @@ static cl::opt<std::string> MemProfImportSummary(
     cl::desc("Import summary to use for testing the ThinLTO backend via opt"),
     cl::Hidden);
 
+static cl::opt<unsigned>
+    TailCallSearchDepth("memprof-tail-call-search-depth", cl::init(5),
----------------
snehasish wrote:

Do you think we need to have an option or a hard coded limit is fine? Given our strategy of selecting only unambiguous paths, increasing depth will increase both runtime and chance of an ambiguous path. So then we'd probably won't tune it and I'd prefer not to introduce a flag which we never change. What do you think?

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


More information about the llvm-commits mailing list