[PATCH] D135720: [clang][deps] Remove more codegen options

2022-10-11 Thread Jan Svoboda 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 rG060b25346b95: [clang][deps] Remove more codegen options (authored by jansvoboda11). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D135720: [clang][deps] Remove more codegen options

2022-10-11 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 added a comment. In D135720#3850954 , @Bigcheese wrote: > lgtm. Although I wonder if long term we should add something to the option > tablegen file for this case? I think a good option would be to have a function that uses the information

[PATCH] D135720: [clang][deps] Remove more codegen options

2022-10-11 Thread Michael Spencer via Phabricator via cfe-commits
Bigcheese accepted this revision. Bigcheese added a comment. This revision is now accepted and ready to land. lgtm. Although I wonder if long term we should add something to the option tablegen file for this case? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://review

[PATCH] D135720: [clang][deps] Remove more codegen options

2022-10-11 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 created this revision. jansvoboda11 added a reviewer: Bigcheese. Herald added a subscriber: ributzka. Herald added a project: All. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Codegen options are typically unuse