On Wed, Apr 4, 2012 at 7:19 AM, Tim Northover <[email protected]> wrote: > On Monday 02 Apr 2012 16:37:43 Tim Northover wrote: >> On Monday 02 Apr 2012 16:13:11 Eli Friedman wrote: >> > On Mon, Apr 2, 2012 at 4:46 AM, Tim Northover <[email protected]> > wrote: >> > > Hi, >> > > >> > > When doing something like this: >> > > >> > > const int *CI; >> > > int *I; >> > > *(test ? I : CI) = 0 >> > > >> > > clang currently doesn't put the combined qualifiers on the output type, >> > > which means that the assignment is allowed. I believe this patch >> > > implements correct C99 semantics (and sane behaviour for non-CVR type >> > > qualifiers). >> > > >> > > Could someone review it please? >> > >> > It would be nice to have tests for both "x ? incomplete : complete" >> > and "z ? complete : incomplete" for the tests involving composite >> > types. >> >> Thanks Eli. I've attached an updated patch (also adds a two-way test for >> the const bit, but no other changes). >> >> Could you commit it if it's OK? I don't have access yet. > > Ping.
r154134. -Eli _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
