[wasm] change reducer order in WASM pipeline to make build predictable again

src/compiler/node-matchers.h

 // Copyright 2014 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_COMPILER_NODE_MATCHERS_H_
 #define V8_COMPILER_NODE_MATCHERS_H_
 
 #include <cmath>
 
 // TODO(turbofan): Move ExternalReference out of assembler.h
@@ skipping 234 matching lines @@
 
   const Left& left() const { return left_; }
   const Right& right() const { return right_; }
 
   bool IsFoldable() const { return left().HasValue() && right().HasValue(); }
   bool LeftEqualsRight() const { return left().node() == right().node(); }
 
  protected:
   void SwapInputs() {
     std::swap(left_, right_);
+    // TODO(tebbi): This modification should notify the reducers using

[Michael Starzinger, 2017/03/02 16:38:35]

It might make sense to also file a bug on the issue tracker regarding this.

+    // BinopMatcher. Alternatively, all reducers (especially value numbering)
+    // could ignore the ordering for commutative binops.
     node()->ReplaceInput(0, left().node());
     node()->ReplaceInput(1, right().node());
   }
 
  private:
   void PutConstantOnRight() {
     if (left().HasValue() && !right().HasValue()) {
       SwapInputs();
     }
   }
@@ skipping 453 matching lines @@
   Node* branch_;
   Node* if_true_;
   Node* if_false_;
 };
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_COMPILER_NODE_MATCHERS_H_