This patch fixes the nasty bug that caused 175.vpr to fail for X86 last night.
authorChris Lattner <sabre@nondot.org>
Fri, 15 Oct 2004 03:19:31 +0000 (03:19 +0000)
committerChris Lattner <sabre@nondot.org>
Fri, 15 Oct 2004 03:19:31 +0000 (03:19 +0000)
commit47eb6567e22ecf87ebeabc566fac6c02c975a9a4
treea847eb309a4f92b0db61adf295b0ba830f4c759b
parent8df6a594d2781777cd25fe2d00818419a8b10e5c
This patch fixes the nasty bug that caused 175.vpr to fail for X86 last night.

The problem occurred when trying to reload this instruction:

MOV32mr %reg2326, 8, %reg2297, 4, %reg2295

The value of reg2326 was available in EBX, so it was reused from there, instead
of reloading it into EDX.

The value of reg2297 was available in EDX, so it was reused from there, instead
of reloading it into EDI.

The value of reg2295 was not available, so we tried reloading it into EBX, its
assigned register.  However, we checked and saw that we already reloaded
something into EBX, so we chose what reg2326 was assigned to (EDX) and reloaded
into that register instead.

Unfortunately EDX had already been used by reg2297, so reloading into EDX
clobbered the value used by the reg2326 operand, breaking the program.

The fix for this is to check that the newly picked register is ok.  In this
case we now find that EDX is already used and try using EDI, which succeeds.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@17006 91177308-0d34-0410-b5e6-96231b3b80d8
lib/CodeGen/VirtRegMap.cpp