================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to