[llvm] [CodeLayout] cache-directed sort: limit max chain size (PR #69039)
Fangrui Song via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 19 13:44:08 PDT 2023
================
@@ -62,6 +62,12 @@ cl::opt<bool> ApplyExtTspWithoutProfile(
"ext-tsp-apply-without-profile",
cl::desc("Whether to apply ext-tsp placement for instances w/o profile"),
cl::init(true), cl::Hidden);
+
+namespace codelayout {
+cl::opt<unsigned>
+ CDMaxChainSize("cdsort-max-chain-size", cl::Hidden, cl::init(128),
----------------
MaskRay wrote:
Thanks! I figured it out.
Other options are still named `cds-*`, but this new one is named `cdsort-max-chain-size`. I think it makes sense to reduce the number of abbreviations. As we have used `cdsort` and various case variants, it makes sense to avoid `cds-*`.
Perhaps land a separate commit to rename existing `cds-*` options to `cdsort-*`?
https://github.com/llvm/llvm-project/pull/69039
More information about the llvm-commits
mailing list