[PATCH] D157150: [Driver] Update BoolOption to handle Visibility. NFC

2023-08-15 Thread Justin Bogner via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0db9dfb19329: [Driver] Update BoolOption to handle Visibility. NFC (authored by bogner). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST A

[PATCH] D157150: [Driver] Update BoolOption to handle Visibility. NFC

2023-08-15 Thread Justin Bogner via Phabricator via cfe-commits
bogner updated this revision to Diff 550224. bogner added a comment. Rebase on top of "Visibility" naming. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157150/new/ https://reviews.llvm.org/D157150 Files: clang/include/clang/Driver/Options.h c

[PATCH] D157150: [Driver] Update BoolOption to handle Visibility. NFC

2023-08-14 Thread Justin Bogner via Phabricator via cfe-commits
bogner updated this revision to Diff 550125. bogner edited the summary of this revision. bogner added a comment. Rebase on top of "DefaultVis" naming Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157150/new/ https://reviews.llvm.org/D157150 Files:

[PATCH] D157150: [Driver] Update BoolOption to handle Visibility. NFC

2023-08-14 Thread Justin Bogner via Phabricator via cfe-commits
bogner updated this revision to Diff 549853. bogner added a comment. Use "ClangOption" rather than "Default" in Options.td, as per the conversation in https://reviews.llvm.org/D157151 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157150/new/ https

[PATCH] D157150: [Driver] Update BoolOption to handle Visibility. NFC

2023-08-13 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay accepted this revision. MaskRay added a comment. This revision is now accepted and ready to land. LGTM once the https://reviews.llvm.org/D157151#4582109 "Default" naming discussion is agreed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D15

[PATCH] D157150: [Driver] Update BoolOption to handle Visibility. NFC

2023-08-04 Thread Justin Bogner via Phabricator via cfe-commits
bogner created this revision. bogner added a reviewer: MaskRay. Herald added a subscriber: mcrosier. Herald added a reviewer: sscalpone. Herald added a project: All. bogner requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This updates the Bo