[llvm] r217693 - [AArch64] Enable post-RA MI scheduler.

[Chad Rosier]

I had my VPN token on hand and was able to get to a machine to push the
patch.  Hopefully, the bots will be green after r217725.

 Chad

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