================
@@ -127,8 +127,12 @@ class PPCallbacks {
   /// \param RelativePath The path relative to SearchPath, at which the include
   /// file was found. This is equal to FileName except for framework includes.
   ///
-  /// \param Imported The module, whenever an inclusion directive was
-  /// automatically turned into a module import or null otherwise.
+  /// \param SuggestedModule The module, whenever an inclusion directive was
+  /// considered to be automatically turned into a module import, or null
----------------
benlangmuir wrote:

"was considered to be" most commonly it means you are describing the result of 
a consideration -- "Clang is considered to be a good compiler" -- rather than 
simply saying that something was considered but without reflecting the final 
judgment.  It's ambiguous here.

Suggestion: "The module suggested for this header, if any. See \p 
ModuleImported for whether this include was translated into a module imported"

https://github.com/llvm/llvm-project/pull/81061
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to