[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