CpuProfiler: do not calculate positions if it is not necessary (BuildCompareInstruction part).

src/hydrogen-instructions.h

Index: src/hydrogen-instructions.h
diff --git a/src/hydrogen-instructions.h b/src/hydrogen-instructions.h
index a9eb57d22d91f5aac14620d4ccbd6e694a8ec26f..54e66e1221a489e2be5e53497b0d1d7364f64e9e 100644
--- a/src/hydrogen-instructions.h
+++ b/src/hydrogen-instructions.h
@@ -1191,6 +1191,13 @@ class HControlInstruction : public HInstruction {
     SetSuccessorAt(1, swap);
   }
 
+  void SetOperandPositions(Zone* zone, SourcePosition left_pos,

[Vyacheslav Egorov (Google), 2015/02/27 13:50:15]

This feels really out of place here, because not all HControlInstruction-s have
2 operands. Some have none, some have 1, etc. 

+                           SourcePosition right_pos) {
+    set_operand_position(zone, 0, left_pos);
+    set_operand_position(zone, 1, right_pos);
+  }
+
+
   DECLARE_ABSTRACT_INSTRUCTION(ControlInstruction)
 };
 
@@ -4266,12 +4273,6 @@ class HCompareNumericAndBranch : public HTemplateControlInstruction<2, 2> {
 
   std::ostream& PrintDataTo(std::ostream& os) const OVERRIDE;  // NOLINT
 
-  void SetOperandPositions(Zone* zone, SourcePosition left_pos,
-                           SourcePosition right_pos) {
-    set_operand_position(zone, 0, left_pos);
-    set_operand_position(zone, 1, right_pos);
-  }
-
   DECLARE_CONCRETE_INSTRUCTION(CompareNumericAndBranch)
 
  private: