Add floor, ceil, round (truncate) instructions for ia32, x64 (if SSE4.1) and

src/compiler/arm/instruction-selector-arm.cc

Index: src/compiler/arm/instruction-selector-arm.cc
diff --git a/src/compiler/arm/instruction-selector-arm.cc b/src/compiler/arm/instruction-selector-arm.cc
index 56b1bf0e56dfe184b3d4d1fca17928d4862205a9..5b1e7a0d2b6e8502e2612a0af9458969b3ab9a68 100644
--- a/src/compiler/arm/instruction-selector-arm.cc
+++ b/src/compiler/arm/instruction-selector-arm.cc
@@ -100,6 +100,10 @@ class ArmOperandGenerator : public OperandGenerator {
       case kArmVmodF64:
       case kArmVnegF64:
       case kArmVsqrtF64:
+      case kArmVfloorF64:
+      case kArmVceilF64:
+      case kArmVroundTruncateF64:
+      case kArmVroundTiesAwayF64:
       case kArmVcvtF32F64:
       case kArmVcvtF64F32:
       case kArmVcvtF64S32:
@@ -115,6 +119,14 @@ class ArmOperandGenerator : public OperandGenerator {
 };
 
 
+static void VisitRRFloat64(InstructionSelector* selector, ArchOpcode opcode,
+                           Node* node) {
+  ArmOperandGenerator g(selector);
+  selector->Emit(opcode, g.DefineAsRegister(node),
+                 g.UseRegister(node->InputAt(0)));
+}
+
+
 static void VisitRRRFloat64(InstructionSelector* selector, ArchOpcode opcode,
                             Node* node) {
   ArmOperandGenerator g(selector);
@@ -826,6 +838,38 @@ void InstructionSelector::VisitFloat64Sqrt(Node* node) {
 }
 
 
+void InstructionSelector::VisitFloat64Floor(Node* node) {
+  if (!CpuFeatures::IsSupported(ARMv8)) {

[Benedikt Meurer, 2014/10/30 11:59:12]

I think this should be DCHECK(CpuFeatures::IsSupported(ARMv8)) instead.

[sigurds, 2014/10/30 12:18:26]

Done.

+    UNSUPPORTED_OPERATOR(node);
+  }
+  VisitRRFloat64(this, kArmVfloorF64, node);
+}
+
+
+void InstructionSelector::VisitFloat64Ceil(Node* node) {
+  if (!CpuFeatures::IsSupported(ARMv8)) {

[Benedikt Meurer, 2014/10/30 11:59:12]

I think this should be DCHECK(CpuFeatures::IsSupported(ARMv8)) instead.

[sigurds, 2014/10/30 12:18:26]

Done.

+    UNSUPPORTED_OPERATOR(node);
+  }
+  VisitRRFloat64(this, kArmVceilF64, node);
+}
+
+
+void InstructionSelector::VisitFloat64RoundTruncate(Node* node) {
+  if (!CpuFeatures::IsSupported(ARMv8)) {

[Benedikt Meurer, 2014/10/30 11:59:12]

I think this should be DCHECK(CpuFeatures::IsSupported(ARMv8)) instead.

[sigurds, 2014/10/30 12:18:26]

Done.

+    UNSUPPORTED_OPERATOR(node);
+  }
+  VisitRRFloat64(this, kArmVroundTruncateF64, node);
+}
+
+
+void InstructionSelector::VisitFloat64RoundTiesAway(Node* node) {
+  if (!CpuFeatures::IsSupported(ARMv8)) {

[Benedikt Meurer, 2014/10/30 11:59:12]

I think this should be DCHECK(CpuFeatures::IsSupported(ARMv8)) instead.

[sigurds, 2014/10/30 12:18:26]

Done.

+    UNSUPPORTED_OPERATOR(node);
+  }
+  VisitRRFloat64(this, kArmVroundTiesAwayF64, node);
+}
+
+
 void InstructionSelector::VisitCall(Node* node) {
   ArmOperandGenerator g(this);
   CallDescriptor* descriptor = OpParameter<CallDescriptor*>(node);
@@ -1139,10 +1183,19 @@ void InstructionSelector::VisitFloat64LessThanOrEqual(Node* node) {
 // static
 MachineOperatorBuilder::Flags
 InstructionSelector::SupportedMachineOperatorFlags() {
-  return MachineOperatorBuilder::kInt32DivIsSafe |
-         MachineOperatorBuilder::kInt32ModIsSafe |
-         MachineOperatorBuilder::kUint32DivIsSafe |
-         MachineOperatorBuilder::kUint32ModIsSafe;
+  MachineOperatorBuilder::Flags flags =
+      MachineOperatorBuilder::kInt32DivIsSafe |
+      MachineOperatorBuilder::kInt32ModIsSafe |
+      MachineOperatorBuilder::kUint32DivIsSafe |
+      MachineOperatorBuilder::kUint32ModIsSafe;
+
+  if (CpuFeatures::IsSupported(ARMv8)) {
+    flags |= MachineOperatorBuilder::kFloat64Floor |
+             MachineOperatorBuilder::kFloat64Ceil |
+             MachineOperatorBuilder::kFloat64RoundTruncate |
+             MachineOperatorBuilder::kFloat64RoundTiesAway;
+  }
+  return flags;
 }
 
 }  // namespace compiler