Np!<br><br>On Saturday, September 13, 2014, Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks Chad!<br>
<br>
On 13 September 2014 04:34, Chad Rosier <<a href="javascript:;" onclick="_e(event, 'cvml', 'mcrosier@codeaurora.org')">mcrosier@codeaurora.org</a>> wrote:<br>
> I had my VPN token on hand and was able to get to a machine to push the<br>
> patch.  Hopefully, the bots will be green after r217725.<br>
><br>
>  Chad<br>
><br>
>> All,<br>
>> If someone wouldn't mind verifying the patch and pushing the change,<br>
>> Sanjin and I would greatly appreciate it.  I won't have access to fix this<br>
>> until Sunday and I'm not sure Sanjin has commit access.<br>
>><br>
>>  Thanks,<br>
>>   Chad<br>
>><br>
>>> Hi Renato and Chad,<br>
>>><br>
>>> Attached is the patch that modifies the test case to use regular<br>
>>> expressions<br>
>>> to check when the post-RA scheduler pass is being skipped, regardless of<br>
>>> the<br>
>>> post-RA scheduler implementation.  I tested this for both ARM (which<br>
>>> used<br>
>>> the post-RA top-down list latency scheduler) and AArch64 (which now uses<br>
>>> the<br>
>>> post-RA MI scheduler).<br>
>>><br>
>>> Thanks,<br>
>>> Sanjin<br>
>>><br>
>>> -----Original Message-----<br>
>>> From: <a href="javascript:;" onclick="_e(event, 'cvml', 'llvm-commits-bounces@cs.uiuc.edu')">llvm-commits-bounces@cs.uiuc.edu</a><br>
>>> [mailto:<a href="javascript:;" onclick="_e(event, 'cvml', 'llvm-commits-bounces@cs.uiuc.edu')">llvm-commits-bounces@cs.uiuc.edu</a>] On Behalf Of Sanjin Sijaric<br>
>>> Sent: Friday, September 12, 2014 4:52 PM<br>
>>> To: 'Renato Golin'; 'Chad Rosier'<br>
>>> Cc: 'LLVM Commits'<br>
>>> Subject: RE: [llvm] r217693 - [AArch64] Enable post-RA MI scheduler.<br>
>>><br>
>>> Hi Renato,<br>
>>><br>
>>> When I did make check-all, it came clean (I see that that<br>
>>> Feature/optnone-llc.ll passes in my logs).   But this was because my<br>
>>> build<br>
>>> was a dual arm + aarch64 build that defaults to arm.  So the checks in<br>
>>> optnone-llc.ll were valid, as the default is the" post-RA top down<br>
>>> latency<br>
>>> scheduler" for the arm configuration.  Now that it's the post-RA MI<br>
>>> scheduler for the  aarch64 configuration, the aarch64-only build will<br>
>>> fail<br>
>>> since what shows up by default is "Skipping pass 'PostRA Machine<br>
>>> Instruction<br>
>>> Scheduler'", and not "Skipping pass 'Post RA top-down list latency<br>
>>> scheduler'".<br>
>>><br>
>>> I'll see if I can change the test case to match the new behaviour.<br>
>>><br>
>>> Sorry about this oversight.<br>
>>><br>
>>> Sanjin<br>
>>><br>
>>> -----Original Message-----<br>
>>> From: <a href="javascript:;" onclick="_e(event, 'cvml', 'llvm-commits-bounces@cs.uiuc.edu')">llvm-commits-bounces@cs.uiuc.edu</a><br>
>>> [mailto:<a href="javascript:;" onclick="_e(event, 'cvml', 'llvm-commits-bounces@cs.uiuc.edu')">llvm-commits-bounces@cs.uiuc.edu</a>] On Behalf Of Renato Golin<br>
>>> Sent: Friday, September 12, 2014 3:32 PM<br>
>>> To: Chad Rosier<br>
>>> Cc: LLVM Commits<br>
>>> Subject: Re: [llvm] r217693 - [AArch64] Enable post-RA MI scheduler.<br>
>>><br>
>>> On 12 September 2014 18:40, Chad Rosier <<a href="javascript:;" onclick="_e(event, 'cvml', 'mcrosier@codeaurora.org')">mcrosier@codeaurora.org</a>> wrote:<br>
>>>> Author: mcrosier<br>
>>>> Date: Fri Sep 12 12:40:39 2014<br>
>>>> New Revision: 217693<br>
>>>><br>
>>>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=217693&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=217693&view=rev</a><br>
>>>> Log:<br>
>>>> [AArch64] Enable post-RA MI scheduler.<br>
>>><br>
>>> Hi Chad,<br>
>>><br>
>>> Have you seen this?<br>
>>><br>
>>> <a href="http://lab.llvm.org:8011/builders/llvm-aarch64-linux/builds/4106" target="_blank">http://lab.llvm.org:8011/builders/llvm-aarch64-linux/builds/4106</a><br>
>>><br>
>>> cheers,<br>
>>> --renato<br>
>>> _______________________________________________<br>
>>> llvm-commits mailing list<br>
>>> <a href="javascript:;" onclick="_e(event, 'cvml', 'llvm-commits@cs.uiuc.edu')">llvm-commits@cs.uiuc.edu</a><br>
>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
>>><br>
>>> _______________________________________________<br>
>>> llvm-commits mailing list<br>
>>> <a href="javascript:;" onclick="_e(event, 'cvml', 'llvm-commits@cs.uiuc.edu')">llvm-commits@cs.uiuc.edu</a><br>
>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
>>><br>
>><br>
>><br>
>><br>
><br>
><br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="javascript:;" onclick="_e(event, 'cvml', 'llvm-commits@cs.uiuc.edu')">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote>