================
@@ -580,6 +585,18 @@ class alignas(IdentifierInfoAlignment) IdentifierInfo {
       RecomputeNeedsHandleIdentifier();
   }
 
+  /// Determine whether this is the contextual keyword \c module.
+  bool isModulesDeclaration() const { return IsModulesDecl; }
+
+  /// Set whether this identifier is the contextual keyword \c module.
+  void setModulesDeclaration(bool I) {
----------------
cor3ntin wrote:

The naming here is pretty terrible

`isModulesImport` is pre-existing (to be read as "is module's import") but it 
sets a bad precedent

I would prefer

`isImportKeyword` and `isModuleKeyword`



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

Reply via email to