[clang] Revise the modules document for clarity (PR #90237)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 26 12:44:39 PDT 2024


================
@@ -998,25 +992,23 @@ headers to:
   ...
   #endif
 
-If the modules imported by your library provides such headers too, remember to add them to
-your ``your_library_imported.h`` too.
+If the modules imported by the library provide such headers, remember to add
+them to ``your_library_imported.h`` too.
 
 Importing modules
 ~~~~~~~~~~~~~~~~~
 
-When there are dependent libraries providing modules, we suggest you to import that in
-your module.
-
-Most of the existing libraries would fall into this catagory once the std module gets available.
+When there are dependent libraries providing modules, they should be imported
+in your module as well. Many existing libraries will fall into this catagory
+once the ``std`` module is more widely available.
 
 All dependent libraries providing modules
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
-Life gets easier if all the dependent libraries providing modules.
+Life gets easier if all the dependent libraries provide modules.
----------------
erichkeane wrote:

```suggestion
Of course, most of the complexity disappears if all the dependent libraries provide modules.
```

https://github.com/llvm/llvm-project/pull/90237


More information about the cfe-commits mailing list