<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, May 19, 2014 at 1:11 PM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><br>
<br>
> On May 19, 2014, at 12:30, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
><br>
> I did a little bit more digging into the issue, and the problem is in<br>
> ARCMT.cpp:318, whic has the code:<br>
><br>
>  Diags->setDiagnosticMapping(diag::err_arcmt_nsalloc_realloc,<br>
>                              NoNSAllocReallocError ? diag::MAP_WARNING<br>
>                                                    : diag::MAP_ERROR,<br>
>                              SourceLocation());<br>
><br>
> You cannot map an error diagnostic to be a warning.<br>
> DiagnosticsEngine::setDiagnosticMapping has this assert, which is<br>
> triggered by that code:<br>
><br>
>  assert((Diags->isBuiltinWarningOrExtension(Diag) ||<br>
>          (Map == diag::MAP_FATAL || Map == diag::MAP_ERROR)) &&<br>
>         "Cannot map errors into warnings!");<br>
><br>
> I'm not familiar enough with ARCMT to understand what is required for<br>
> the correct fix.<br>
<br>
</div>If the current behavior is correct, the right way to declare the diagnostic in the .td file is "Warning<...>, DefaultError".</blockquote><div><br></div><div>... and in that case, its name should not begin err_. </div>
</div></div></div>