Use a bigger hammer to fix PR11314 by disabling the "forcing two-address
authorEvan Cheng <evan.cheng@apple.com>
Thu, 10 Nov 2011 07:43:16 +0000 (07:43 +0000)
committerEvan Cheng <evan.cheng@apple.com>
Thu, 10 Nov 2011 07:43:16 +0000 (07:43 +0000)
commit623a7e146bd86747dc46a6f8bb9993fc217d6b78
tree72b44ba84e3cb347279c411ac8b159e6fecaf9b0
parent4dbe96e22ff4989577cf4854f717b9d972094f87
Use a bigger hammer to fix PR11314 by disabling the "forcing two-address
instruction lower optimization" in the pre-RA scheduler.

The optimization, rather the hack, was done before MI use-list was available.
Now we should be able to implement it in a better way, perhaps in the
two-address pass until a MI scheduler is available.

Now that the scheduler has to backtrack to handle call sequences. Adding
artificial scheduling constraints is just not safe. Furthermore, the hack
is not taking all the other scheduling decisions into consideration so it's just
as likely to pessimize code. So I view disabling this optimization goodness
regardless of PR11314.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@144267 91177308-0d34-0410-b5e6-96231b3b80d8
12 files changed:
lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp
test/CodeGen/X86/2006-05-11-InstrSched.ll
test/CodeGen/X86/2009-04-21-NoReloadImpDef.ll
test/CodeGen/X86/change-compare-stride-1.ll
test/CodeGen/X86/fold-pcmpeqd-0.ll
test/CodeGen/X86/iv-users-in-other-loops.ll
test/CodeGen/X86/lsr-loop-exit-cond.ll
test/CodeGen/X86/lsr-reuse-trunc.ll
test/CodeGen/X86/masked-iv-safe.ll
test/CodeGen/X86/multiple-loop-post-inc.ll
test/CodeGen/X86/sse2.ll
test/CodeGen/X86/sse3.ll