<div dir="ltr">On Thu, Sep 26, 2013 at 1:57 PM, Marshall Clow <span dir="ltr"><<a href="mailto:mclow.lists@gmail.com" target="_blank" class="cremed">mclow.lists@gmail.com</a>></span> wrote:<br><div class="gmail_extra">
<div class="gmail_quote"><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">While discussing the status of <strstream> in C++14, I decided to search through the LLVM/clang code base for uses of strstream.<br>

<br>
I didn't find any, just a header file that includes <strstream>.<br>
<br>
Here's a patch that removes that. This should (marginally) speed up builds.<br></blockquote><div><br></div><div>I don't see in that in your patch. I do see this:</div><div><br></div><div><div>Index: INPUTS/all-std-headers.cpp</div>
<div>===================================================================</div><div>--- INPUTS/all-std-headers.cpp<span class="" style="white-space:pre">    </span>(revision 191429)</div><div>+++ INPUTS/all-std-headers.cpp<span class="" style="white-space:pre">    </span>(working copy)</div>
<div>@@ -44,9 +44,9 @@</div><div> #include <stdexcept></div><div> #include <streambuf></div><div> #include <string></div><div>-#if __has_include(<strstream>)</div><div>-#include <strstream></div>
<div>-#endif</div><div>+// #if __has_include(<strstream>)</div><div>+// #include <strstream></div><div>+// #endif</div><div> #include <typeinfo></div><div> #include <utility></div><div> #include <valarray></div>
</div><div><br></div><div>Please don't apply this part. We want to be able to test that Clang succeeds at parsing all headers in the standard library, so if it has strstream, we should test it. And we want to benchmark it, for the same reasons.</div>
<div><br></div><div>This file isn't actually part of any build anyways.</div><div> </div><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">

<br>
Also, I removed a mention of <strstream> from a test for "has_header" - I used <new> instead.<br></blockquote><div><br></div><div>This is fine. </div><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">

<br>
-- Marshall<br>
<br>
Marshall Clow     Idio Software   <mailto:<a href="mailto:mclow.lists@gmail.com" class="cremed">mclow.lists@gmail.com</a>><br>
<br>
A.D. 1517: Martin Luther nails his 95 Theses to the church door and is promptly moderated down to (-1, Flamebait).<br>
        -- Yu Suzuki<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" class="cremed">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div></div>