================
@@ -50,3 +50,16 @@ Options
 
   - `UseStatic`
     Add ``static`` for internal linkage variable and function.
+
+.. option:: AnalyzeFunctions
+
+  Whether to suggest giving functions internal linkage. Default is `true`.
+
+.. option:: AnalyzeVariables
+
+  Whether to suggest giving variables internal linkage. Default is `true`.
+
+.. option:: AnalyzeTypes
+
+  (C++-only) Whether to suggest giving user-defined types (structs,
----------------
localspook wrote:

```suggestion
  (C++ only) Whether to suggest giving user-defined types (structs,
```
That's fair, I've gone and removed the hyphen. As for natural language: with 
the way it is right now, a C user, upon seeing this option, can *immediately* 
tell that it isn't relevant to them. Phrasing it more naturally would probably 
require them to read a bit more before coming to that understanding, which I 
feel would be a bit of a downgrade.

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

Reply via email to