Remove unnecessary checks for forced context allocation

src/ast/scopes.cc

Index: src/ast/scopes.cc
diff --git a/src/ast/scopes.cc b/src/ast/scopes.cc
index 4e107bd3fdab1a258a5ff747ebc63893094cd5e8..7a430f74e57d98e7fbf2ca48e995c8d08d60d396 100644
--- a/src/ast/scopes.cc
+++ b/src/ast/scopes.cc
@@ -1577,7 +1577,7 @@ void DeclarationScope::AllocateParameterLocals() {
     if (var == rest_parameter_) continue;
 
     DCHECK(var->scope() == this);
-    if (uses_sloppy_arguments || has_forced_context_allocation()) {
+    if (uses_sloppy_arguments) {
       // Force context allocation of the parameter.

[neis, 2016/08/09 07:15:26]

Maybe remove the comment too as it doesn't add any information.

[adamk, 2016/08/09 16:29:11]

Done.

       var->ForceContextAllocation();
     }
@@ -1607,11 +1607,6 @@ void DeclarationScope::AllocateReceiver() {
   if (!has_this_declaration()) return;
   DCHECK_NOT_NULL(receiver());
   DCHECK_EQ(receiver()->scope(), this);
-
-  if (has_forced_context_allocation()) {
-    // Force context allocation of the receiver.
-    receiver()->ForceContextAllocation();
-  }
   AllocateParameter(receiver(), -1);
 }