Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-08-18 Thread Hans Wennborg via cfe-commits
, 2017 at 4:35 PM Shoaib Meenai wrote: >> >> >> >> See Petr's follow-up. It doesn't look like this was the responsible >> >> change. >> >> >> >> >> >> >> >> From: Eric Fiselier >> >> Date:

Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-08-18 Thread Eric Fiselier via cfe-commits
e responsible > >> change. > >> > >> > >> > >> From: Eric Fiselier > >> Date: Thursday, August 17, 2017 at 4:02 PM > >> To: Petr Hosek > >> Cc: cfe-commits , Marshall Clow > >> , Hans Wennborg , Shoaib &g

Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-08-18 Thread Hans Wennborg via cfe-commits
t; To: Petr Hosek >> Cc: cfe-commits , Marshall Clow >> , Hans Wennborg , Shoaib Meenai >> >> Subject: Re: [libcxx] r309474 - [libc++] Hoist extern template above first >> use >> >> >> >> probably shouldn't merge this then. >&g

Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-08-17 Thread Petr Hosek via cfe-commits
t; > > > *From: *Eric Fiselier > *Date: *Thursday, August 17, 2017 at 4:02 PM > *To: *Petr Hosek > *Cc: *cfe-commits , Marshall Clow < > mclow.li...@gmail.com>, Hans Wennborg , Shoaib Meenai < > smee...@fb.com> > *Subject: *Re: [libcxx] r309474 - [libc++] Hoist

Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-08-17 Thread Shoaib Meenai via cfe-commits
See Petr's follow-up. It doesn't look like this was the responsible change. From: Eric Fiselier Date: Thursday, August 17, 2017 at 4:02 PM To: Petr Hosek Cc: cfe-commits , Marshall Clow , Hans Wennborg , Shoaib Meenai Subject: Re: [libcxx] r309474 - [libc++] Hoist extern template a

Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-08-17 Thread Eric Fiselier via cfe-commits
probably shouldn't merge this then. On Aug 17, 2017 4:54 PM, "Petr Hosek via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > This broke our build of WebKit with the following build failure: > > ../../buildtools/linux-x64/clang/bin/clang++ -MD -MF > obj/apps/web_view/web_view_test.test_webview

Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-08-17 Thread Shoaib Meenai via cfe-commits
"cfe-commits@lists.llvm.org" Subject: Re: [libcxx] r309474 - [libc++] Hoist extern template above first use Actually nevermind, it seems like it was a different change and this one is harmless (I'm still bisecting libc++ to find out which is causing the error below). On Thu, Aug

Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-08-17 Thread Petr Hosek via cfe-commits
Actually nevermind, it seems like it was a different change and this one is harmless (I'm still bisecting libc++ to find out which is causing the error below). On Thu, Aug 17, 2017 at 3:54 PM Petr Hosek wrote: > This broke our build of WebKit with the following build failure: > > ../../buildtool

Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-08-17 Thread Petr Hosek via cfe-commits
This broke our build of WebKit with the following build failure: ../../buildtools/linux-x64/clang/bin/clang++ -MD -MF obj/apps/web_view/web_view_test.test_webview.o.d -DTOOLCHAIN_VERSION=4e89c701396412a50a901115ab4a2a09145f3777 -D_LIBCPP_ENABLE_THREAD_SAFETY_ANNOTATIONS -DCAIRO_HAS_FC_FONT=0 -DU_U

Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-08-09 Thread Hans Wennborg via cfe-commits
Marshall: ping? On Thu, Aug 3, 2017 at 10:20 AM, Hans Wennborg wrote: > Sounds good to me, but Eric or Marshall need to sign off. > > On Thu, Aug 3, 2017 at 10:15 AM, Shoaib Meenai wrote: >> Ping. >> >> On 7/28/17, 7:57 PM, "Shoaib Meenai" wrote: >> >> Marshall, Eric, Hans, >> >> Any ob

Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-08-03 Thread Hans Wennborg via cfe-commits
Sounds good to me, but Eric or Marshall need to sign off. On Thu, Aug 3, 2017 at 10:15 AM, Shoaib Meenai wrote: > Ping. > > On 7/28/17, 7:57 PM, "Shoaib Meenai" wrote: > > Marshall, Eric, Hans, > > Any objections to backporting this to 5.0? It fixes a potential visibility > issue for

Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-08-03 Thread Shoaib Meenai via cfe-commits
Ping. On 7/28/17, 7:57 PM, "Shoaib Meenai" wrote: Marshall, Eric, Hans, Any objections to backporting this to 5.0? It fixes a potential visibility issue for clients of the header. On 7/28/17, 7:54 PM, "cfe-commits on behalf of Shoaib Meenai via cfe-commits" wrote:

Re: [libcxx] r309474 - [libc++] Hoist extern template above first use

2017-07-28 Thread Shoaib Meenai via cfe-commits
Marshall, Eric, Hans, Any objections to backporting this to 5.0? It fixes a potential visibility issue for clients of the header. On 7/28/17, 7:54 PM, "cfe-commits on behalf of Shoaib Meenai via cfe-commits" wrote: Author: smeenai Date: Fri Jul 28 19:54:41 2017 New Revision: 30947