[llvm] 88a2031 - [FuncSpec] Fix typo in option description. NFC.

Sjoerd Meijer via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 8 04:58:53 PDT 2021


Author: Sjoerd Meijer
Date: 2021-09-08T12:58:46+01:00
New Revision: 88a203120790ff273d545c6f8f05e2b66c5ed9e3

URL: https://github.com/llvm/llvm-project/commit/88a203120790ff273d545c6f8f05e2b66c5ed9e3
DIFF: https://github.com/llvm/llvm-project/commit/88a203120790ff273d545c6f8f05e2b66c5ed9e3.diff

LOG: [FuncSpec] Fix typo in option description. NFC.

Added: 
    

Modified: 
    llvm/lib/Transforms/IPO/FunctionSpecialization.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/IPO/FunctionSpecialization.cpp b/llvm/lib/Transforms/IPO/FunctionSpecialization.cpp
index 757efe2e8dd81..03a318e734765 100644
--- a/llvm/lib/Transforms/IPO/FunctionSpecialization.cpp
+++ b/llvm/lib/Transforms/IPO/FunctionSpecialization.cpp
@@ -68,8 +68,8 @@ static cl::opt<unsigned> MaxConstantsThreshold(
 
 static cl::opt<unsigned> SmallFunctionThreshold(
     "func-specialization-size-threshold", cl::Hidden,
-    cl::desc("For functions whose IR instruction count below this threshold, "
-             " they wouldn't be specialized to avoid useless sepcializations."),
+    cl::desc("Don't specialize functions that have less than this theshold "
+             "number of instructions"),
     cl::init(100));
 
 static cl::opt<unsigned>
@@ -77,9 +77,12 @@ static cl::opt<unsigned>
                           cl::desc("Average loop iteration count cost"),
                           cl::init(10));
 
+// TODO: This needs checking to see the impact on compile-times, which is why
+// this is off by default for now.
 static cl::opt<bool> EnableSpecializationForLiteralConstant(
     "function-specialization-for-literal-constant", cl::init(false), cl::Hidden,
-    cl::desc("Make function specialization available for literal constant."));
+    cl::desc("Enable specialization of functions that take a literal constant "
+             "as an argument."));
 
 // Helper to check if \p LV is either a constant or a constant
 // range with a single element. This should cover exactly the same cases as the


        


More information about the llvm-commits mailing list