[PATCH] D99351: [CSSPGO] Top-down processing order based on full profile.

[Wei Mi via Phabricator via llvm-commits]

wmi added a comment.

The performance test result is neutral, so I think we can enable UseProfiledCallGraph by default.



================
Comment at: llvm/lib/Transforms/IPO/SampleProfile.cpp:165
+static cl::opt<bool>
+    UseProfiledCallGraph("use-profiled-call-graph", cl::init(false), cl::Hidden,
+                         cl::desc("Process functions in a top-down order "
----------------
Emit an error to prevent misuse if ProfileTopDownLoad is false and UseProfiledCallGraph is true?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99351/new/

https://reviews.llvm.org/D99351