<p dir="ltr"><br>
On Aug 22, 2013 9:11 PM, "Doug Gregor" <<a href="mailto:dgregor@apple.com">dgregor@apple.com</a>> wrote:<br>
><br>
><br>
><br>
> ================<br>
> Comment at: lib/Sema/SemaTemplate.cpp:4177<br>
> @@ +4176,3 @@<br>
> +    SourceLocation FirstOpLoc;<br>
> +    while (UnaryOperator *UnOp = dyn_cast<UnaryOperator>(Arg)) {<br>
> +      UnaryOperatorKind UnOpKind = UnOp->getOpcode();<br>
> ----------------<br>
> Do we really want to strip of all of these &/* operators without even an extension warning?</p>
<p dir="ltr">Agreed. I've said it in other contexts, but to reiterate: when adding compatibility for non standard features like this, we need extension warnings. When the features are actively broken by design for some inputs, we need a "known buggy" warning (I'm looking at you record layout.... ::shudder:: )<br>
<br></p>
<p dir="ltr">><br>
><br>
> <a href="http://llvm-reviews.chandlerc.com/D1444">http://llvm-reviews.chandlerc.com/D1444</a><br>
><br>
> BRANCH<br>
>   PR16905<br>
><br>
> ARCANIST PROJECT<br>
>   clang<br>
> _______________________________________________<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">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</p>