<div dir="rtl"><div dir="ltr">Hi Eugene,</div><div dir="ltr"><br></div><div dir="ltr">This patch fails the unittests which also need to be updated to reflect the changes, see</div><div dir="ltr"><br></div><div dir="ltr"><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lab.llvm.org-3A8011_builders_clang-2Dx86-5F64-2Ddebian-2Dfast_builds_28343&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=clj2R9BSvZRDZjuj83PAILcPhJapmrY4ZVarD8A9yfo&s=z0wPZXtsMWhxUKQcSkNwErNowBrK6XC23vIzyT_T9SY&e=">http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/28343</a><br></div><div dir="ltr"><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lab.llvm.org-3A8011_builders_clang-2Dcmake-2Dthumbv7-2Da15_builds_3820&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=clj2R9BSvZRDZjuj83PAILcPhJapmrY4ZVarD8A9yfo&s=qiJ6GUe9QqP1xIdweVAuVQs_ZimriHHfYvcjfOTn_dw&e=">http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-a15/builds/3820</a><br></div><div dir="ltr"><br></div><div dir="ltr">Yaron</div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2015-07-02 23:24 GMT+03:00 Eugene Kosov <span dir="ltr"><<a href="mailto:claprix@yandex.ru" target="_blank">claprix@yandex.ru</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Commit them, please. I have no commit rights.<br>
<br>
Thanks for the help.<br>
<br>
02.07.2015, 22:44, "Yaron Keren" <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>>:<br>
<div><div class="h5">> Hi Eugene,<br>
><br>
> These patches LGTM. Should I commit them for you?<br>
><br>
> Yaron<br>
><br>
> 2015-06-26 23:33 GMT+03:00 Eugene Kosov <<a href="mailto:claprix@yandex.ru">claprix@yandex.ru</a>>:<br>
>> Hi.<br>
>><br>
>> Thanks for your feedback. I fixed the regression. Here is the new patch.<br>
>><br>
>> 26.06.2015, 00:51, "Yaron Keren" <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>>:<br>
>><br>
>>> Hi Eugene,<br>
>>><br>
>>> This patch does not pass clang extra tools regression tests locally,<br>
>>><br>
>>> FAIL: Clang Tools :: modularize/NoProblems.modularize (7141 of 19860)<br>
>>> Testing: 0 .. 10.. 20.. 30<br>
>>> FAIL: Clang Tools :: modularize/NoProblemsDependencies.modularize (7144 of 19860)<br>
>>> Testing: 0 .. 10.. 20.. 30<br>
>>> FAIL: Clang Tools :: modularize/NoProblemsCoverage.modularize (7145 of 19860)<br>
>>> (...)<br>
>>><br>
>>> Please check it out.<br>
>>><br>
>>> Yaron<br>
>>><br>
>>> 2015-06-22 15:25 GMT+03:00 Eugene Kosov <<a href="mailto:claprix@yandex.ru">claprix@yandex.ru</a>>:<br>
>>>> Hi.<br>
>>>><br>
>>>> Here are two patches for clang-tools-extra and for clang which simplifies code a little bit and also removes possibility of implicit std::string construction.<br>
>>>><br>
>>>> --<br>
>>>> Eugene<br>
>>>><br>
>>>> _______________________________________________<br>
>>>> cfe-commits mailing list<br>
>>>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
>><br>
>> --<br>
>> Eugene<br>
<br>
</div></div>--<br>
Eugene<br>
</blockquote></div><br></div>