Re: [PATCH] D12037: WindowsX86: long double is x87DoubleExtended on mingw-w64

2015-08-14 Thread Hans Wennborg via cfe-commits
hans added a comment. I'm OK with it if Richard gives owner's approval. Cheers, Hans http://reviews.llvm.org/D12037 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12037: WindowsX86: long double is x87DoubleExtended on mingw-w64

2015-08-14 Thread Hans Wennborg via cfe-commits
I'm OK with it if Richard gives owner's approval. Cheers, Hans On Fri, Aug 14, 2015 at 12:25 PM, Martell Malone wrote: > martell added a comment. > > Hans this is something we most definitely should have in 3.7 > It's a pretty big bugfix for this target :) > > > http://reviews.llvm.org/D12037 >

Re: [PATCH] D12037: WindowsX86: long double is x87DoubleExtended on mingw-w64

2015-08-14 Thread Martell Malone via cfe-commits
martell added a comment. Hans this is something we most definitely should have in 3.7 It's a pretty big bugfix for this target :) http://reviews.llvm.org/D12037 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/ma

Re: [PATCH] D12037: WindowsX86: long double is x87DoubleExtended on mingw-w64

2015-08-14 Thread Martell Malone via cfe-commits
martell added a comment. Landed in http://reviews.llvm.org/rL245084 The commit parser is bugged :/ http://reviews.llvm.org/D12037 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12037: WindowsX86: long double is x87DoubleExtended on mingw-w64

2015-08-14 Thread Reid Kleckner via cfe-commits
rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. lgtm http://reviews.llvm.org/D12037 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits