diff options
| author | Rafael Espindola <rafael.espindola@gmail.com> | 2009-03-28 17:03:24 +0000 |
|---|---|---|
| committer | Rafael Espindola <rafael.espindola@gmail.com> | 2009-03-28 17:03:24 +0000 |
| commit | c2a17d30221ec873c360aedb967e1ed6c07e951d (patch) | |
| tree | 2852d8fa23fcb7f1b5308475ec6ccf2949ace9fa /llvm/lib/Target/X86/X86FloatingPoint.cpp | |
| parent | 351c71a85f74eea1f8e68d7efbe506f8cb06fcf6 (diff) | |
| download | bcm5719-llvm-c2a17d30221ec873c360aedb967e1ed6c07e951d.tar.gz bcm5719-llvm-c2a17d30221ec873c360aedb967e1ed6c07e951d.zip | |
Make code a bit less brittle by no hardcoding the number
of operands in an address in so many places.
llvm-svn: 67945
Diffstat (limited to 'llvm/lib/Target/X86/X86FloatingPoint.cpp')
| -rw-r--r-- | llvm/lib/Target/X86/X86FloatingPoint.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/Target/X86/X86FloatingPoint.cpp b/llvm/lib/Target/X86/X86FloatingPoint.cpp index 8862428b226..dee57176d9f 100644 --- a/llvm/lib/Target/X86/X86FloatingPoint.cpp +++ b/llvm/lib/Target/X86/X86FloatingPoint.cpp @@ -616,9 +616,10 @@ void FPS::handleZeroArgFP(MachineBasicBlock::iterator &I) { /// handleOneArgFP - fst <mem>, ST(0) /// void FPS::handleOneArgFP(MachineBasicBlock::iterator &I) { + const int X86AddrNumOperands = 4; MachineInstr *MI = I; unsigned NumOps = MI->getDesc().getNumOperands(); - assert((NumOps == 5 || NumOps == 1) && + assert((NumOps == X86AddrNumOperands + 1 || NumOps == 1) && "Can only handle fst* & ftst instructions!"); // Is this the last use of the source register? |
