aengelke wrote: Looks reasonable, but why? If all OptionCategory instances are const+constinit, they just move from .data to .data.rel.ro due to StringRef initializers, changing effectively nothing in practice, except for a fair amount of code churn.
(I would be somewhat hesitant to make larger changes to cl::opt until there's a consensus on which direction we want to take w.r.t. multi-threaded tool execution -- not that we can't do it, but it might not be worth the effort or wasted effort if we end up taking a different direction.) https://github.com/llvm/llvm-project/pull/184352 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
