[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