<div dir="ltr">Hi Sylvestre,<div><br></div><div>I agree with this change, too many people still believe that -O20 will give them 7x better code than -O3... sigh...</div><div><br></div><div>Some comments below:<br><div class="gmail_extra">
<br><div class="gmail_quote">On 8 November 2013 06:05, Sylvestre Ledru <span dir="ltr"><<a href="mailto:sylvestre@debian.org" target="_blank">sylvestre@debian.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
   unsigned OptLevel = getOptimizationLevel(Args, IK, Diags);<br>
   if (OptLevel > 3) {<br>
-    Diags.Report(diag::err_drv_invalid_value)<br>
-      << Args.getLastArg(OPT_O)->getAsString(Args) << OptLevel;<br>
-    OptLevel = 3;<br>
-    Success = false;<br>
+    unsigned DefaultOptLevel = 3;<br>
+    Diags.Report(diag::warn_drv_invalid_value)<br>
+        << Args.getLastArg(OPT_O)->getAsString(Args) << OptLevel << DefaultOptLevel;<br>
+    OptLevel = DefaultOptLevel;<br>
   }<br>
   Opts.OptimizationLevel = OptLevel;<br></blockquote><div></div></div><br></div></div><div class="gmail_extra">This seems slightly redundant. Either you use literal 3 all over, or set the DefaultOptLevel on a higher place. Also, this is not the "default" level, but the fall-back level for high levels, maybe a better word would be "MaxOptLevel", if there isn't one already (I don't know).</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">If there isn't, than maybe you could create a MaxOptLevel on a higher level and change all comparisons "OptLevel > 3" for "OptLevel > MaxOptLevel".</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">cheers,</div><div class="gmail_extra">--renato</div></div>