dexonsmith added a comment.

In D83892#2458603 <https://reviews.llvm.org/D83892#2458603>, @dexonsmith wrote:

> Maybe it's worth defining another set of derived multiclasses, [...]

Another possibility of course is to make `Bool*Option` work this way (removing 
the ChangedBy, ResetBy, DefaultsToTrue, and DefaultsToFalse helpers).

(Either way, if we do add better support for dynamic defaults, I think the 
names `SetsTo{True,False}` are more clear than my previous example / suggestion 
`{True,False}Flag`.)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83892/new/

https://reviews.llvm.org/D83892

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to