rjmccall added a comment.

In https://reviews.llvm.org/D29739#674288, @probinson wrote:

> In https://reviews.llvm.org/D29739#673971, @rjmccall wrote:
>
> > In https://reviews.llvm.org/D29739#673933, @tigerleapgorge wrote:
> >
> > > Hi John,
> > >
> > > Here is the most recent discussion I can find on cfe-dev.
> > >  “I'm guessing that Objective-C/C++ is kind of passe, so nobody is really 
> > > interested in modernizing it”
> > >  http://lists.llvm.org/pipermail/cfe-dev/2016-December/051844.html
> > >
> > > As far as I am aware, there appears to be no strong reason to bump or not 
> > > to bump ObjC++.
> >
> >
> > It certainly simplifies the message to say that we've changed the default 
> > C++ dialect to C++11 across the board.  That should apply to ObjC++ as 
> > well.  I would not describe ObjC++ as passé; it's a very important language 
> > for Apple developers.
>
>
> Nice to know, although nobody piped up on the earlier cited discussion.
>
> Sony is invested in making the lit tests C++11 clean so that we can upstream 
> a change to make it the default C++ dialect for PS4.  That will ensure that 
> any new C++ tests are C++11 clean.  This is one step in the direction of 
> making C++11 (or even something newer) the default dialect for everybody.
>  However we are not an Objective-C++ vendor.  We are neutral about changing 
> the default dialect there; if you want to change the default dialect for 
> Objective-C++, that's fine with us, but I don't think we can invest in 
> learning enough about Objective-C++ to do the right thing with the existing 
> Objective-C++ tests.  In particular I don't know whether forcing 98 on these 
> tests is the "right" solution; all we know is that it made the tests pass, 
> which is not particularly surprising.


I understand.  That's part of why I do code review, because sometimes I can 
answer questions for other people. :)  Forcing 98 on these tests is fine.

> I really think Apple would need to step up here if the default Objective-C++ 
> dialect is going to change.

I don't mind stepping up and doing this work.  I just thought you'd already 
done it.  This patch updates some tests; is that enough, or are there further 
changes required in order to change the default ObjC++ dialect?

John.


https://reviews.llvm.org/D29739



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to