<div dir="ltr">On Thu, Feb 27, 2014 at 6:23 PM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</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"><div class="HOEnZb"><div class="h5">On 02/26/2014 10:27 PM, Tobias Grosser wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 02/26/2014 10:19 PM, Chris Lattner wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Feb 26, 2014, at 2:22 AM, Tobias Grosser <<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br></blockquote></blockquote>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
3) How to enable 'remarks'<br>
<br>
We need a way to enable 'remark' diagnostics. Quentin proposed to go<br>
for an approach similar to the warning flags. Where we control remarks<br>
with '-Rvector', '-Rloop-vector', ...<br>
<br>
I will read a little bit through the existing option system to better<br>
understand what it is doing, possibly adding documentation / cleanups<br>
on my way. I will come back with a proposal here.<br>
</blockquote>
<br>
It’s a bit odd, but since these are diagnostics, why not use the<br>
existing -W flags?  You should be able to -Werror one of these,<br>
control them with #pragma clang diagnostics, etc.  It doesn’t seem<br>
like we need more complexity in this space.<br>
</blockquote>
<br>
Good point. I will prepare the above patches such that they reuse the<br>
existing infrastructure. If we really see a need for further<br>
adjustments, we can do this incrementally.<br>
</blockquote>
<br></div></div>
I just updated the patch to reflect the conclusions of our discussion.<br>
Please review for commit.<br>
<br>
------------<br>
[PATCH] Add 'remark' diagnostic type in 'clang'<div class="im"><br>
<br>
A 'remark' is information that is not an error or a warning, but rather some additional information provided to the user. In contrast to a 'note' a 'remark' is an independent diagnostic, whereas a 'note' always depends on another diagnostic.<br>

<br>
A typical use case for remark nodes is information provided to the user, e.g. information provided by the vectorizer about loops that have been vectorized.<br>
<br></div>
This patch provides the initial implementation of 'remarks'. It includes the actual definiton of the remark nodes, their printing as well as basic parameter handling. We are reusing the existing diagnostic parameters which means a remark<br>

can be enabled with normal '-Wdiagnostic-name' flags and can be upgraded to an error using '-Werror=diagnostic-name'.<br></blockquote><div><br></div><div>For the record, I strongly recommend that the syntax to enable a remark should be "-Wremark=diagnostic-name", and that "-Wdiagnostic-name" should (continue to) mean "upgrade this diagnostic to a warning".</div>
<div>However, I have no objection to the current semantics for an initial checkin, as long as nobody uses it as an excuse to keep those semantics forever.</div><div><br></div><div>my $.02,</div><div>–Arthur</div></div></div>
</div>