<div dir="ltr"><div>As an example of other compilers, icc can emit diagnostics like this (with <span style="color:rgb(51,51,51);font-family:'Courier New',Courier,monospace;font-size:13.333333015441895px;line-height:16px">-vec-report2 </span>flag<span style="color:rgb(51,51,51);font-family:'Courier New',Courier,monospace;font-size:13.333333015441895px;line-height:16px">):</span><br>
</div><div><br></div><div>s.cpp(42): (col. 9) remark: loop was not vectorized: existence of vector dependence</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-02-25 15:29 GMT+04:00 Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com" target="_blank">clattner@apple.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
On Feb 25, 2014, at 2:41 AM, Tobias Grosser <<a href="mailto:tobias@grosser.es">tobias@grosser.es</a>> wrote:<br>
<br>
> On 02/25/2014 11:32 AM, David Majnemer wrote:<br>
>> You've added some extra whitespace in the definition<br>
>> of CXDiagnosticSeverity.<br>
>> Perhaps more importantly, the enum isn't in increasing order anymore.<br>
><br>
> Hi David,<br>
><br>
> thanks for the fast review.<br>
><br>
> I fixed the whitespace issue.<br>
><br>
> The order was intentional. I wanted to list the fields sorted by severity, because this is how they are sorted logically. However, we need to use '= 5' for the new diagnostic to not break the ABI here (these are the c bindings). Would you propose another way to write this enum?<br>

<br>
</div></div>Is "remark" better than "informative" diagnostics?  Is there prior art in other compilers that would be worth following?<br>
<span class="HOEnZb"><font color="#888888"><br>
-Chris<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Best regards,<br>Alexander Musman<br>+79154687051<br>skype: alexander.musman<br><div><br></div>
</div>