[cfe-commits] r133526 - /cfe/trunk/lib/Sema/SemaExpr.cpp

Chandler Carruth chandlerc at gmail.com
Tue Jun 21 11:34:46 PDT 2011


On Tue, Jun 21, 2011 at 11:23 AM, Douglas Gregor <dgregor at apple.com> wrote:

> I'm not a big fan of this change. In general, we've taken the approach that
> the "silence the warning" option is always first.


Ahh. Is that documented anywhere? It's not enforced by tests that I can
see... Anyways, didn't realize there was any policy.


I actually find this fairly unfortunate across the board. Most of the cases
where we have an alternative to silencing the warning, it's a more likely
alternative. It'd seem nice to invert the policy (silence comes last), but
maybe that can't be changed at this late stage.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110621/d609f512/attachment.html>


More information about the cfe-commits mailing list