Sorry, it's based on a few patches i have locally that I didn't put up for review but can't commit yet<br><div class="gmail_quote"><div dir="ltr">On Tue, Aug 30, 2016 at 6:13 AM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added a comment.<br>
<br>
Which revision is this patch against? It does not seem to apply cleanly to trunk for me. Could you try running the test suite on linux or mac as well, as the last StringRef change left things in a pretty broken state there (you are changing code which is pretty-much unused on windows)? If not I can give it a shot once it the merge issues are resolved.<br>
<br>
Apart from that, I like the direction this is going in.<br>
<br>
<br>
================<br>
Comment at: unittests/Utility/StringExtractorTest.cpp:76<br>
@@ +75,3 @@<br>
+    EXPECT_EQ("ExtractorTest::Consume()", ex.Peek());<br>
+    ;<br>
+}<br>
----------------<br>
This seems unnecessary.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D24013" rel="noreferrer" target="_blank">https://reviews.llvm.org/D24013</a><br>
<br>
<br>
<br>
</blockquote></div>