Skip to content

V
V8
Commit 1b84cc2c authored by sampsong's avatar sampsong Committed by Commit Bot
Browse files
Options

PPC: Remove unnecessary frsp before stfs in codegen 

BUG=

R=jyan@ca.ibm.com, bjaideep@ca.ibm.com, joransiu@ca.ibm.com

Review-Url: https://codereview.chromium.org/2905453002
Cr-Commit-Position: refs/heads/master@{#45609}
parent f56a48e8
Hide whitespace changes
Inline Side-by-side
Showing with 8 additions and 5 deletions
src/compiler/ppc/code-generator-ppc.cc
View file @ 1b84cc2c
......@@ -587,11 +587,12 @@ Condition FlagsConditionToCondition(FlagsCondition condition, ArchOpcode op) {
AddressingMode mode = kMode_None; \
MemOperand operand = i.MemoryOperand(&mode, &index); \
DoubleRegister value = i.InputDoubleRegister(index); \
__ frsp(kScratchDoubleReg, value); \
/* removed frsp as instruction-selector checked */ \
/* value to be kFloat32 */ \
if (mode == kMode_MRI) { \
__ stfs(kScratchDoubleReg, operand); \
__ stfs(value, operand); \
} else { \
__ stfsx(kScratchDoubleReg, operand); \
__ stfsx(value, operand); \
} \
DCHECK_EQ(LeaveRC, i.OutputRCBit()); \
} while (0)
......@@ -699,11 +700,13 @@ Condition FlagsConditionToCondition(FlagsCondition condition, ArchOpcode op) {
__ bge(&done); \
DoubleRegister value = i.InputDoubleRegister(3); \
__ frsp(kScratchDoubleReg, value); \
/* removed frsp as instruction-selector checked */ \
/* value to be kFloat32 */ \
if (mode == kMode_MRI) { \
__ stfs(kScratchDoubleReg, operand); \
__ stfs(value, operand); \
} else { \
CleanUInt32(offset); \
__ stfsx(kScratchDoubleReg, operand); \
__ stfsx(value, operand); \
} \
__ bind(&done); \
DCHECK_EQ(LeaveRC, i.OutputRCBit()); \
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment