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

[Mitch Bodart via llvm-commits]

mbodart added a comment.

Thanks for the testing suggestions Sanjay, but I think I'll stick with the current test for now.

Regarding 1), that would only test that some other scheduling paradigm did not separate the loads, which may or may not be a desired behavior, and may change from time to time.  So I prefer to limit the test to getting the expected behavior in the known settings where it is desired.

As for 2), the default cpu varies by tool.  With clang -m32 it is pentium4 and with clang -m64 it is x86_64,
but with llc it is generic.  So the test would indeed fail if I added an llc RUN case with no explicit cpu.


http://reviews.llvm.org/D19138