On Aug 22, 2013 9:11 PM, "Doug Gregor" <[email protected]> wrote: > > > > ================ > Comment at: lib/Sema/SemaTemplate.cpp:4177 > @@ +4176,3 @@ > + SourceLocation FirstOpLoc; > + while (UnaryOperator *UnOp = dyn_cast<UnaryOperator>(Arg)) { > + UnaryOperatorKind UnOpKind = UnOp->getOpcode(); > ---------------- > Do we really want to strip of all of these &/* operators without even an extension warning?
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:: ) > > > http://llvm-reviews.chandlerc.com/D1444 > > BRANCH > PR16905 > > ARCANIST PROJECT > clang > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
