<div dir="rtl"><div dir="ltr">Committed revision 241433.<br></div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2015-07-05 0:30 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">Hi.<br>
<br>
Here is the patch which fixes clang unit tests build. I built llvm without tests - sorry for that.<br>
<br>
03.07.2015, 12:42, "Yaron Keren" <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>>:<br>
<div><div class="h5">> Hi Eugene,<br>
><br>
> This patch fails the unittests which also need to be updated to reflect the changes, see<br>
><br>
> <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=wuri_ApfqxOePkWIZzVPPPMHJb8NFcjCXeFnGGJNHyU&s=SI3PZLTQ4N1xOtvisZnoZhb8hAZQ4P7uTAUHcYlH3Vo&e=" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/28343</a><br>
> <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=wuri_ApfqxOePkWIZzVPPPMHJb8NFcjCXeFnGGJNHyU&s=U3ed9LrxxZvCzZITrL2nIaFyMal_QY_ZaIY4D-Od0w4&e=" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-a15/builds/3820</a><br>
><br>
> Yaron<br>
><br>
> 2015-07-02 23:24 GMT+03:00 Eugene Kosov <<a href="mailto:claprix@yandex.ru">claprix@yandex.ru</a>>:<br>
>> 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>
>><br>
>>> 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>
>> --<br>
>> Eugene<br>
<br>
</div></div>--<br>
Eugene<br>
</blockquote></div><br></div>