Allocation site pretenuring.

src/hydrogen.cc

Index: src/hydrogen.cc
diff --git a/src/hydrogen.cc b/src/hydrogen.cc
index b6498b1dbbce8786fa2d021df5196e7e274e0df6..bdd2f43c54dc897f6180f520455b226f564119a9 100644
--- a/src/hydrogen.cc
+++ b/src/hydrogen.cc
@@ -9344,16 +9344,10 @@ HInstruction* HOptimizedGraphBuilder::BuildFastLiteral(
   HValue* object_size_constant = Add<HConstant>(
       boilerplate_object->map()->instance_size());
 
-  // We should pull pre-tenure mode from the allocation site.
-  // For now, just see what it says, and remark on it if it sez
-  // we should pretenure. That means the rudimentary counting in the garbage
-  // collector is having an effect.
-  PretenureFlag pretenure_flag = isolate()->heap()->GetPretenureMode();
-  if (FLAG_allocation_site_pretenuring) {
-    pretenure_flag = site_context->current()->GetPretenureMode()
-        ? TENURED
-        : NOT_TENURED;
-  }
+  PretenureFlag pretenure_flag = site_context->current()->GetPretenureMode();
+
+  site_context->current()->AddDependentCompilationInfo(

[Michael Starzinger, 2013/12/02 15:12:54]

IMHO both accesses here should be behind the FLAG_allocation_site_pretenuring
flag. If that flag is false, we should either fall-back to the global mode or to
NOT_TENURED I guess. WDYT?

Unless you are planning to remove that flag completely soon, then we could just
start dropping checks already.

[Hannes Payer (out of office), 2014/01/09 14:37:13]

Done. Makes sense to have both flags for some time.

+      AllocationSite::TENURING, top_info());
 
   HInstruction* object = Add<HAllocate>(object_size_constant, type,
       pretenure_flag, instance_type, site_context->current());