[PATCH] D82756: Port some floating point options to new option marshalling infrastructure

Jan Svoboda via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 27 06:47:40 PDT 2020


jansvoboda11 added a subscriber: Anastasia.
jansvoboda11 added a comment.

Using the records with `DefaultAnyOf<[...]>` to catch errors seems to work well, thanks for the suggestion.

(I was initially thinking about using the keypaths to avoid errors and reorder the options in the TableGen backend to make everything work.)

@dexonsmith WDYT about the implementation?

@Anastasia There is no need to manually expand `-cc1` options anymore. Do you have any other concerns?


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

https://reviews.llvm.org/D82756



More information about the llvm-commits mailing list