<div dir="ltr">On Fri, Sep 27, 2013 at 8:36 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><div class="gmail_extra">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">There are some special -f options that are not really flags. For<br>
example, -fprofile-dir=PATH. It takes a value and there is no such<br>
thing as -fno-profile-dir. For these special cases it looks like we<br>
have a fairly reasonable infrastructure, since they will need some<br>
custom handling anyway.<br>
<br>
But the vast majority are just simple flags that have the form of the<br>
pair -ffoo and -fno-foo. Right now we have quiet a large amount of<br>
boilerplate to implement those.<br>
<br>
* Add both to Options.td:<br>
<br>
def fshow_column : Flag<["-"], "fshow-column">, Group<f_Group>,<br>
Flags<[CC1Option]>;<br>
def fno_show_column : Flag<["-"], "fno-show-column">, Group<f_Group>,<br>
Flags<[CC1Option]>,<br>
  HelpText<"Do not include column number on diagnostics">;<br>
<br>
* Add it to some .def file.<br>
<br>
DIAGOPT(ShowColumn, 1, 1)       /// Show column number on diagnostics.<br></blockquote><div><br></div><div class="gmail_quote">I don't think this part of it is really an issue... this plus the bit of code in lib/Frontend is sort of boilerplate, but I don't think it's worth increasing the scope of the patch to cover it.  I would lean towards keeping the options in the driver.<br>
<br></div></div><div class="gmail_quote">-Eli<br></div></div></div>