[Turbofan] Streamline InstructionOperand predicate methods.

src/machine-type.h

Index: src/machine-type.h
diff --git a/src/machine-type.h b/src/machine-type.h
index 062e15d9c9a9d7a668d695d166f4b4b36655183e..705df13a691db04de698c3498a1a729716fa61df 100644
--- a/src/machine-type.h
+++ b/src/machine-type.h
@@ -22,12 +22,13 @@ enum class MachineRepresentation : uint8_t {
   kWord16,
   kWord32,
   kWord64,
-  kFloat32,
-  kFloat64,  // must follow kFloat32
-  kSimd128,  // must follow kFloat64
   kTaggedSigned,
   kTaggedPointer,
-  kTagged
+  kTagged,
+  // FP representations must be last, and in order of increasing size.

[Mircea Trofin, 2016/09/30 21:52:10]

consider adding a marker here (kFirstFloat) and then the comparisons are
from it

[bbudge, 2016/10/03 21:37:18]

kFirstFPRepresentation

Done.

+  kFloat32,
+  kFloat64,
+  kSimd128
 };
 
 const char* MachineReprToString(MachineRepresentation);
@@ -224,9 +225,7 @@ std::ostream& operator<<(std::ostream& os, MachineSemantic type);
 std::ostream& operator<<(std::ostream& os, MachineType type);
 
 inline bool IsFloatingPoint(MachineRepresentation rep) {
-  return rep == MachineRepresentation::kFloat32 ||
-         rep == MachineRepresentation::kFloat64 ||
-         rep == MachineRepresentation::kSimd128;
+  return rep >= MachineRepresentation::kFloat32;
 }
 
 inline bool CanBeTaggedPointer(MachineRepresentation rep) {