================
@@ -1833,8 +1833,13 @@ class Preprocessor {
   bool LexModuleNameContinue(Token &Tok, SourceLocation UseLoc,
                              SmallVectorImpl<Token> &Suffix,
                              SmallVectorImpl<IdentifierLoc> &Path,
-                             bool AllowMacroExpansion = true,
-                             bool IsPartition = false);
+                             bool IsPartition = false,
+                             bool AllowMacroExpansion = true);
----------------
ojhunt wrote:

I'd rather avoid making this change at the same time - if you do want 
consistency between the functions I'd maintain the existing ordering simply due 
to the fragility of boolean based APIs :-/

If the expectation is that no one is meant to be calling LexModuleNameContinue 
directly anymore I'd remove the default args at least.

https://github.com/llvm/llvm-project/pull/184014
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to