<div dir="ltr"><div class="gmail_extra">> <span style="font-size:12.8px">It would probably make </span><span style="font-size:12.8px">sense for libc++ to remove -Wno-pedantic before removing -pedantic.</span></div><div class="gmail_extra"><span style="font-size:12.8px"><br></span></div><div class="gmail_extra"><span style="font-size:12.8px">Good localized fix. Committed as r</span><span style="font-size:12.8px">250452. `remove_flags` hack needs to die soon :(</span></div><div class="gmail_extra"><span style="font-size:12.8px"><br></span><div class="gmail_quote">On Thu, Oct 15, 2015 at 2:00 PM, Evgenii Stepanov <span dir="ltr"><<a href="mailto:eugeni.stepanov@gmail.com" target="_blank">eugeni.stepanov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">It looks like, since libc++ removes -pedantic, we don't need to add<br>
-Wno-pedantic, so I can just fix it in MSan. It would probably make<br>
sense for libc++ to remove -Wno-pedantic before removing -pedantic.<br>
<br>
On Thu, Oct 15, 2015 at 12:57 PM, Evgenii Stepanov<br>
<div><div><<a href="mailto:eugeni.stepanov@gmail.com" target="_blank">eugeni.stepanov@gmail.com</a>> wrote:<br>
> Transforms it to -Wno.<br>
><br>
> On Thu, Oct 15, 2015 at 12:56 PM, Eric Fiselier <<a href="mailto:eric@efcs.ca" target="_blank">eric@efcs.ca</a>> wrote:<br>
>> Removes it because "-Wno-pedantic" incorrectly matches -pedantic? If my<br>
>> guess is correct, whoops :-(<br>
>><br>
>> On Thu, Oct 15, 2015 at 1:51 PM, Evgenii Stepanov<br>
>> <<a href="mailto:eugeni.stepanov@gmail.com" target="_blank">eugeni.stepanov@gmail.com</a>> wrote:<br>
>>><br>
>>> Guess what this does to the -Wno-pedantic flag added by MSan? :)<br>
>>><br>
>>> On Tue, Oct 13, 2015 at 4:56 PM, Eric Fiselier via cfe-commits<br>
>>> <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
>>> > Author: ericwf<br>
>>> > Date: Tue Oct 13 18:56:33 2015<br>
>>> > New Revision: 250256<br>
>>> ><br>
>>> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=250256&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=250256&view=rev</a><br>
>>> > Log:<br>
>>> > Workaround -pedantic flag added by LLVM<br>
>>> ><br>
>>> > Modified:<br>
>>> >     libcxx/trunk/CMakeLists.txt<br>
>>> ><br>
>>> > Modified: libcxx/trunk/CMakeLists.txt<br>
>>> > URL:<br>
>>> > <a href="http://llvm.org/viewvc/llvm-project/libcxx/trunk/CMakeLists.txt?rev=250256&r1=250255&r2=250256&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/libcxx/trunk/CMakeLists.txt?rev=250256&r1=250255&r2=250256&view=diff</a><br>
>>> ><br>
>>> > ==============================================================================<br>
>>> > --- libcxx/trunk/CMakeLists.txt (original)<br>
>>> > +++ libcxx/trunk/CMakeLists.txt Tue Oct 13 18:56:33 2015<br>
>>> > @@ -198,6 +198,10 @@ include(HandleLibcxxFlags)<br>
>>> >  remove_flags(-DNDEBUG -UNDEBUG -D_DEBUG<br>
>>> >               -stdlib=libc++ -stdlib=libstdc++ -lc++abi -m32)<br>
>>> ><br>
>>> > +# FIXME(EricWF): See the FIXME on LIBCXX_ENABLE_PEDANTIC.<br>
>>> > +# Remove the -pedantic flag provided by LLVM.<br>
>>> > +remove_flags(-pedantic)<br>
>>> > +<br>
>>> >  # Required flags<br>
>>> > ==============================================================<br>
>>> >  add_compile_flags_if_supported(-std=c++11)<br>
>>> >  if (NOT MSVC AND NOT LIBCXX_SUPPORTS_STD_EQ_CXX11_FLAG)<br>
>>> ><br>
>>> ><br>
>>> > _______________________________________________<br>
>>> > cfe-commits mailing list<br>
>>> > <a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
>>> > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
>><br>
>><br>
</div></div></blockquote></div><br></div></div>