<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Feb 23, 2014 at 1:21 AM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 02/23/2014 10:08 AM, tsett wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Tobias,<br>
I got the problem.<br>
I use the DiagnostigBuilder::<u></u>setForceEmit() for this.<br>
An instance of DiagnostigBuilder will be returned if you use the<br>
report-function.<br>
</blockquote>
<br></div>
Good point. The following patch makes my messages appear again:<br>
<br>
--- a/lib/CodeGen/CodeGenAction.<u></u>cpp<br>
+++ b/lib/CodeGen/CodeGenAction.<u></u>cpp<br>
@@ -384,7 +384,9 @@ void BackendConsumer::<u></u>DiagnosticHandlerImpl(const DiagnosticInfo &DI) {<br>
<br>
   // Report the backend message using the usual diagnostic mechanism.<br>
   FullSourceLoc Loc;<br>
-  Diags.Report(Loc, DiagID).AddString(MsgStorage);<br>
+  DiagnosticBuilder DB = Diags.Report(Loc, DiagID);<br>
+  DB.setForceEmit();<br>
+  DB.AddString(MsgStorage);<br>
<br>
We could use this to either force printing of all backend diagnostics or only the notes. However, I am afraid that would break existing flags to suppress diagnostics.<br></blockquote><div><br></div><div>Please don't do this. Notes are designed to attach additional information to the prior non-note diagnostic, and this assumption is pervasive within the diagnostics mechanism inside clang, as well as in the model we present to our users and in our API.</div>
<div><br></div><div>Adding an 'info' or 'remark' level for diagnostics seems much more in line with our existing design. It may be of somewhat limited utility, but the cost of having this support seems rather low. (In the long term, I assume that end users of polly will not care about these diagnostics, and that they're primarily to aid people developing or testing polly -- and certainly this does not match how other optimization passes in LLVM report debug or progress information to their users.) FWIW, some other compilers have a 'remark' diagnostic level, for messages that don't rise to the level of 'warning'.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Another option is to only set this for plugin diagnostics. However, this would not solve the problem for the vectorizers which may need similar reporting capabilities.<div class="HOEnZb"><div class="h5"><br>
<br>
Tobias<br>
______________________________<u></u>_________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/cfe-dev</a><br>
</div></div></blockquote></div><br></div></div>