RE: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets.

2017-03-01 Thread Robinson, Paul via cfe-commits
: Wednesday, March 01, 2017 3:52 PM To: Sean Silva Cc: Robinson, Paul; cfe-commits Subject: Re: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets. On Mar 1, 2017, at 3:24 PM, Sean Silva mailto:chisophu...@gmail.com>> wrote: On Wed, Mar 1, 2017 at 10:35 AM, Mehdi

Re: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets.

2017-03-01 Thread Mehdi Amini via cfe-commits
> On Mar 1, 2017, at 3:24 PM, Sean Silva wrote: > > > > On Wed, Mar 1, 2017 at 10:35 AM, Mehdi Amini > wrote: > I’m not sure I find this nice to see this upstream. > > I not fond in general of this kind of difference in behavior. I don’t think > it is good for

Re: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets.

2017-03-01 Thread Sean Silva via cfe-commits
On Wed, Mar 1, 2017 at 10:35 AM, Mehdi Amini wrote: > I’m not sure I find this nice to see this upstream. > > I not fond in general of this kind of difference in behavior. I don’t > think it is good for clang to have different default for this kind of > settings depending on the platform. It does

RE: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets.

2017-03-01 Thread Robinson, Paul via cfe-commits
Wednesday, March 01, 2017 10:36 AM To: Sean Silva Cc: Robinson, Paul; cfe-commits Subject: Re: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets. I’m not sure I find this nice to see this upstream. I not fond in general of this kind of difference in behavior. I don’t think it

Re: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets.

2017-03-01 Thread Mehdi Amini via cfe-commits
I’m not sure I find this nice to see this upstream. I not fond in general of this kind of difference in behavior. I don’t think it is good for clang to have different default for this kind of settings depending on the platform. It does not provide a very good user experience from a cross-platfo

Re: r296554 - [PS4] Set our default dialect to C++11. NFC for other targets.

2017-02-28 Thread Sean Silva via cfe-commits
Nice! -- Sean Silva On Tue, Feb 28, 2017 at 5:01 PM, Paul Robinson via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: probinson > Date: Tue Feb 28 19:01:10 2017 > New Revision: 296554 > > URL: http://llvm.org/viewvc/llvm-project?rev=296554&view=rev > Log: > [PS4] Set our default dial