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

I've changed it to:
HelpTextForVariants (for a single entry in the list, which is many variants to 
one string)
Help**Texts**ForVariants (for the list itself, which is many 
HelpTextForVariants)

Which is bound to confuse someone, but it's that or have them be confused why 
HelpTextForVariant is actually a many to one mapping which seems worse.

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