Changes in directory llvm/lib/Target/X86:
X86CodeEmitter.cpp updated: 1.87 -> 1.88 --- Log message: Bug fix: XS, XD prefixes were being emitted twice. XMM registers were not being handled. --- Diffs of the changes: (+9 -9) X86CodeEmitter.cpp | 18 +++++++++--------- 1 files changed, 9 insertions(+), 9 deletions(-) Index: llvm/lib/Target/X86/X86CodeEmitter.cpp diff -u llvm/lib/Target/X86/X86CodeEmitter.cpp:1.87 llvm/lib/Target/X86/X86CodeEmitter.cpp:1.88 --- llvm/lib/Target/X86/X86CodeEmitter.cpp:1.87 Wed Feb 1 00:13:50 2006 +++ llvm/lib/Target/X86/X86CodeEmitter.cpp Tue Feb 14 15:45:24 2006 @@ -186,6 +186,11 @@ case X86::ST0: case X86::ST1: case X86::ST2: case X86::ST3: case X86::ST4: case X86::ST5: case X86::ST6: case X86::ST7: return RegNo-X86::ST0; + + case X86::XMM0: case X86::XMM1: case X86::XMM2: case X86::XMM3: + case X86::XMM4: case X86::XMM5: case X86::XMM6: case X86::XMM7: + return RegNo-X86::XMM0; + default: assert(MRegisterInfo::isVirtualRegister(RegNo) && "Unknown physical register!"); @@ -371,21 +376,16 @@ case X86II::TB: MCE.emitByte(0x0F); // Two-byte opcode prefix break; - case X86II::REP: break; // already handled. - case X86II::XS: // F3 0F - MCE.emitByte(0xF3); - MCE.emitByte(0x0F); - break; - case X86II::XD: // F2 0F - MCE.emitByte(0xF2); - MCE.emitByte(0x0F); - break; case X86II::D8: case X86II::D9: case X86II::DA: case X86II::DB: case X86II::DC: case X86II::DD: case X86II::DE: case X86II::DF: MCE.emitByte(0xD8+ (((Desc.TSFlags & X86II::Op0Mask)-X86II::D8) >> X86II::Op0Shift)); break; // Two-byte opcode prefix + case X86II::REP: + case X86II::XS: + case X86II::XD: + break; // already handled. default: assert(0 && "Invalid prefix!"); case 0: break; // No prefix! } _______________________________________________ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits