From 3a34c35f953b5fe1bd6c4621072fb9c862748d6f Mon Sep 17 00:00:00 2001
From: Jakob Stoklund Olesen <jolesen@mozilla.com>
Date: Mon, 2 Oct 2017 14:43:30 -0700
Subject: [PATCH] Don't swap the arguments to the select instruction.

Both WebAssembly and Cretonne use true-value, false-value.
---
 lib/wasm/src/code_translator.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/wasm/src/code_translator.rs b/lib/wasm/src/code_translator.rs
index 39a1d27253..24eae598a2 100644
--- a/lib/wasm/src/code_translator.rs
+++ b/lib/wasm/src/code_translator.rs
@@ -95,7 +95,7 @@ pub fn translate_operator<FE: FuncEnvironment + ?Sized>(
         }
         Operator::Select => {
             let (arg1, arg2, cond) = state.pop3();
-            state.push1(builder.ins().select(cond, arg2, arg1));
+            state.push1(builder.ins().select(cond, arg1, arg2));
         }
         Operator::Nop => {
             // We do nothing