[PATCH] D157149: [Option] Add "Visibility" field and clone the OptTable APIs to use it

Justin Bogner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 14 17:06:58 PDT 2023


bogner added inline comments.


================
Comment at: llvm/include/llvm/Option/OptParser.td:153
 class Flags<list<OptionFlag> flags> { list<OptionFlag> Flags = flags; }
+class Vis<list<OptionVisibility> vis> { list<OptionVisibility> Vis = vis; }
 class Group<OptionGroup group> { OptionGroup Group = group; }
----------------
phosek wrote:
> Would it be possible to spell this out in full, that is `Visibility` rather than `Vis`? It's more consistent with all other attributes and improves readability in my opinion.
Sure, I can change it. I went with brevity thinking that it would help readability, especially in clang's Options.td, but I'm fairly okay with either way.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157149



More information about the llvm-commits mailing list