[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