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
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
