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

[Sanjin Sijaric]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: PATCH
Type: application/octet-stream
Size: 1151 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140912/e01bf330/attachment.obj>