[PATCH] D83892: [clang][cli] Port CodeGen option flags to new option parsing system

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 26 01:04:18 PST 2020


jansvoboda11 added a comment.

This is now ready to be reviewed. If you see a deleted option in the diff, it's either because it was moved closer to its counterpart, or because it's now generated by a multiclass.
I've added a bunch of TODOs I plan to address in a future patch, most of them aim to compress two options into a single multiclass that describes their relationship (e.g. `OptOutPositiveFlag`).
I'd like to make `OptInFFlag` et al. modular with something like `OptInFlag` that does not imply `Flags<[CC1Option]>` and `Group<f_Group>` and can be marked with `IsPositive` or `IsNegative`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83892



More information about the cfe-commits mailing list