ahatanak added inline comments.
================ Comment at: include/clang/Driver/Options.td:1419 +def fno_disable_tail_calls_escaping_blocks : Flag<["-"], "fno-disable-tail-calls-escaping-blocks">, Group<f_Group>, Flags<[CC1Option]>; +def fdisable_tail_calls_escaping_blocks : Flag<["-"], "fdisable-tail-calls-escaping-blocks">, Group<f_Group>, Flags<[CC1Option]>; def force__cpusubtype__ALL : Flag<["-"], "force_cpusubtype_ALL">; ---------------- rjmccall wrote: > rjmccall wrote: > > These are pretty unidiomatic option names. I would suggest one of these: > > - [fixed]-fescaping-block-tail-calls[/fixed] (the default) and > > [fixed]-fno-escaping-block-tail-calls[/fixed] > > - [fixed]-enable-escaping-block-tail-calls[/fixed] (the default) and > > [fixed]-disable-escaping-block-tail-calls[/fixed] > Wow, this is not even close to Phabricator markup, I don't know what I was > thinking. I chose the first option "-fno-escaping-block-tail-calls/-fescaping-block-tail-calls". https://reviews.llvm.org/D43841 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
