From 4cf15e7a3bed2ff3a7fa87f18cdf65b7eecccee2 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Sun, 11 Apr 2004 20:21:06 +0000 Subject: [PATCH] Relax assertion to make this function work with a broader class of instructions git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@12836 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/X86/FloatingPoint.cpp | 10 ++++++++-- lib/Target/X86/X86FloatingPoint.cpp | 10 ++++++++-- 2 files changed, 16 insertions(+), 4 deletions(-) diff --git a/lib/Target/X86/FloatingPoint.cpp b/lib/Target/X86/FloatingPoint.cpp index b1a2ceb79c5..6afa7e76cfb 100644 --- a/lib/Target/X86/FloatingPoint.cpp +++ b/lib/Target/X86/FloatingPoint.cpp @@ -449,11 +449,17 @@ void FPS::handleOneArgFP(MachineBasicBlock::iterator &I) { } -/// handleOneArgFPRW - fchs - ST(0) = -ST(0) +/// handleOneArgFPRW: Handle instructions that read from the top of stack and +/// replace the value with a newly computed value. These instructions may have +/// non-fp operands after their FP operands. +/// +/// Examples: +/// R1 = fchs R2 +/// R1 = fadd R2, [mem] /// void FPS::handleOneArgFPRW(MachineBasicBlock::iterator &I) { MachineInstr *MI = I; - assert(MI->getNumOperands() == 2 && "Can only handle fst* instructions!"); + assert(MI->getNumOperands() >= 2 && "FPRW instructions must have 2 ops!!"); // Is this the last use of the source register? unsigned Reg = getFPReg(MI->getOperand(1)); diff --git a/lib/Target/X86/X86FloatingPoint.cpp b/lib/Target/X86/X86FloatingPoint.cpp index b1a2ceb79c5..6afa7e76cfb 100644 --- a/lib/Target/X86/X86FloatingPoint.cpp +++ b/lib/Target/X86/X86FloatingPoint.cpp @@ -449,11 +449,17 @@ void FPS::handleOneArgFP(MachineBasicBlock::iterator &I) { } -/// handleOneArgFPRW - fchs - ST(0) = -ST(0) +/// handleOneArgFPRW: Handle instructions that read from the top of stack and +/// replace the value with a newly computed value. These instructions may have +/// non-fp operands after their FP operands. +/// +/// Examples: +/// R1 = fchs R2 +/// R1 = fadd R2, [mem] /// void FPS::handleOneArgFPRW(MachineBasicBlock::iterator &I) { MachineInstr *MI = I; - assert(MI->getNumOperands() == 2 && "Can only handle fst* instructions!"); + assert(MI->getNumOperands() >= 2 && "FPRW instructions must have 2 ops!!"); // Is this the last use of the source register? unsigned Reg = getFPReg(MI->getOperand(1));