678681 - IonMonkey: Implement TestVAndBranch

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[David Anderson [:dvander] - inactive, e-mail if emergency]

We're failing a bunch of tests because of this and it shouldn't be hard to implement.

Comment 1

[David Anderson [:dvander] - inactive, e-mail if emergency]

Attachment: patch

Comment 2

[Sean Stangl [:sstangl]]

Comment on attachment 555670 [details] [diff] [review]
patch

Review of attachment 555670 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/ion/shared/MacroAssembler-x86-shared.h
@@ +78,5 @@
>          jmp(label);
>      }
>  
> +    // cond: NonZero = truthy, Zero = falsey
> +    Condition testDoubleTruthy(Condition cond, const FloatRegister &reg) {

JS_ASSERT(cond == Assembler::NonZero || cond == Assembler::Zero);

Or even better:

Condition testDoubleTruthinessIs(bool truthiness, const FloatRegister &reg) {

::: js/src/ion/x64/CodeGenerator-x64.cpp
@@ +176,5 @@
> +    return ScratchReg;
> +}
> +
> +Assembler::Condition
> +CodeGeneratorX64::testStringTruthy(Assembler::Condition cond, const ValueOperand &value)

bool

::: js/src/ion/x64/MacroAssembler-x64.h
@@ +232,5 @@
>          loadDouble(*dp, dest);
>      }
> +
> +    // Equal = true, NotEqual = false
> +    Condition testInt32Truthy(Condition cond, const ValueOperand &operand) {

bool truthiness,

@@ +237,5 @@
> +        JS_ASSERT(cond == Zero || cond == NonZero);
> +        testl(operand.valueReg(), operand.valueReg());
> +        return cond;
> +    }
> +    Condition testBooleanTruthy(Condition cond, const ValueOperand &operand) {

bool truthiness,

::: js/src/ion/x86/CodeGenerator-x86.cpp
@@ +234,5 @@
> +    return value.typeReg();
> +}
> +
> +Assembler::Condition
> +CodeGeneratorX86::testStringTruthy(Assembler::Condition cond, const ValueOperand &value)

testStringTruthiness(bool truthiness, const ValueOperand &value) {

::: js/src/ion/x86/MacroAssembler-x86.h
@@ +190,5 @@
>          movsd(dp, dest);
>      }
> +
> +    // cond: NonZero = truthy, Zero = falsey
> +    Condition testInt32Truthy(Condition cond, const ValueOperand &operand) {

bool

@@ +195,5 @@
> +        JS_ASSERT(cond == Zero || cond == NonZero);
> +        testl(operand.payloadReg(), operand.payloadReg());
> +        return cond;
> +    }
> +    Condition testBooleanTruthy(Condition cond, const ValueOperand &operand) {

bool

Comment 3

[David Anderson [:dvander] - inactive, e-mail if emergency]

http://hg.mozilla.org/projects/ionmonkey/rev/c9779aee0822 w/ nits