[llvm] 02f20a0 - [Option] Remove the error-prone default argument true from 4-argument hasFlag

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 26 01:09:23 PDT 2022


Author: Fangrui Song
Date: 2022-03-26T01:09:18-07:00
New Revision: 02f20a09c3ae1d9763d835ebae1c025ac819a079

URL: https://github.com/llvm/llvm-project/commit/02f20a09c3ae1d9763d835ebae1c025ac819a079
DIFF: https://github.com/llvm/llvm-project/commit/02f20a09c3ae1d9763d835ebae1c025ac819a079.diff

LOG: [Option] Remove the error-prone default argument true from 4-argument hasFlag

Added: 
    

Modified: 
    llvm/include/llvm/Option/ArgList.h

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/Option/ArgList.h b/llvm/include/llvm/Option/ArgList.h
index e19ecec573226..c9dd7da24e80e 100644
--- a/llvm/include/llvm/Option/ArgList.h
+++ b/llvm/include/llvm/Option/ArgList.h
@@ -305,7 +305,7 @@ class ArgList {
   /// the negation is present, and \p Default if none of the options are
   /// given. If multiple options are present, the last one wins.
   bool hasFlag(OptSpecifier Pos, OptSpecifier PosAlias, OptSpecifier Neg,
-               bool Default = true) const;
+               bool Default) const;
 
   /// Render only the last argument match \p Id0, if present.
   template<typename ...OptSpecifiers>


        


More information about the llvm-commits mailing list