[runtime] Unify and simplify how frames are marked

src/compiler/pipeline.cc

Index: src/compiler/pipeline.cc
diff --git a/src/compiler/pipeline.cc b/src/compiler/pipeline.cc
index 61cecab53cd6cdf583d291d1aabcb4304d2b6773..21361cbe0bfffcdfd31385af3484440fd4cd3c83 100644
--- a/src/compiler/pipeline.cc
+++ b/src/compiler/pipeline.cc
@@ -280,10 +280,10 @@ class PipelineData {
     DCHECK(frame_ == nullptr);
     int fixed_frame_size = 0;
     if (descriptor != nullptr) {
-      fixed_frame_size = (descriptor->IsCFunctionCall())
-                             ? StandardFrameConstants::kFixedSlotCountAboveFp +
-                                   StandardFrameConstants::kCPSlotCount
-                             : StandardFrameConstants::kFixedSlotCount;
+      fixed_frame_size = (((descriptor->flags() &

[Benedikt Meurer, 2016/02/22 18:16:38]

Nit: For "flags" I'd prefer to not use explicit comparison against 0 to
distinguish them easily from integer bitmasking.

[danno, 2016/03/07 09:33:38]

Done.

+                            CallDescriptor::kHasStandardFrameHeader) != 0)
+                              ? StandardFrameConstants::kFixedSlotCount
+                              : TypedFrameConstants::kFixedSlotCount);
     }
     frame_ = new (instruction_zone()) Frame(fixed_frame_size, descriptor);
   }