[flang-commits] [clang] [clang-tools-extra] [flang] [lld] [llvm] [flang][clang] Add Visibility specific help text for options (PR #81869)

Andrzej WarzyƄski via flang-commits flang-commits at lists.llvm.org
Mon Mar 11 11:16:42 PDT 2024


================
@@ -93,6 +93,11 @@ class OptionGroup<string name> {
 
 // Define the option class.
 
+class HelpTextForVisibility<OptionVisibility visibility, string text> {
----------------
banach-space wrote:

Naming is hard :) How about `HelpTextVariant`? Basically, `HelpTextForVisibility` refers to the implementation detail ("visibility") and I suggest that the name reflects the functionality provided.

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


More information about the flang-commits mailing list