[PATCH] D19138: [X86] Enable the post-RA-scheduler for 32-bit cpus

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 27 15:27:01 PDT 2016


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM.

A couple of options to make sure the behavior is as expected:

1. To confirm the codegen difference with -post-RA-scheduler, you could add a RUN (and corresponding CHECKs) that disables -post-RA-scheduler for one of the tested CPUs. Or add a run line for a newer CPU that does not have -post-RA-scheduler turned on by default.

2. To confirm that the default i386 CPU model  has -post-RA-scheduler enabled, you could add a RUN line that doesn't explicitly specify the CPU. Then if someone comes along and changes the default i386 target to something not in the current list, it should cause this test to fail.


http://reviews.llvm.org/D19138





More information about the llvm-commits mailing list