On Wed, Aug 28, 2013 at 8:19 PM, Richard Smith <[email protected]> wrote: > Digging into this further, I think this is actually a non-functional change. > There's no real difference between ProcessNonInheritableDeclAttr and > ProcessInheritableDeclAttr.
There is because combining the two cause a lot of test case failures. But I would be *delighted* if we could get rid of the two because it would make consolidated error checking much, much easier. ~Aaron > > > On Wed, Aug 28, 2013 at 5:11 PM, Eli Friedman <[email protected]> > wrote: >> >> On Wed, Aug 28, 2013 at 4:23 PM, Aaron Ballman <[email protected]> >> wrote: >>> >>> Would this make a suitable test case? >>> >>> typedef int i16_1 __attribute((mode(HI))); >>> typedef i16_1 i16_redecl; >> >> >> That isn't a redeclaration. :) >> >> Anyway, I'm not sure you actually can write a meaningful testcase; the >> ModeAttr AST node doesn't actually have any semantics. >> >> -Eli > > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
