[PATCH] D82756: Port some floating point options to new option marshalling infrastructure
Duncan P. N. Exon Smith via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 22 12:09:11 PDT 2020
dexonsmith added a comment.
In D82756#2348159 <https://reviews.llvm.org/D82756#2348159>, @dexonsmith wrote:
> I have an idea: use `DEFAULT_VALUE` to keep current behaviour. Here's an example to demonstrate.
Downside of this is that it's error prone, because it requires for correctness (but does not enforce) a particular option definition order. The alternative would be to formalize it in the options somehow.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82756/new/
https://reviews.llvm.org/D82756
More information about the llvm-commits
mailing list