Comment about why we don't record evals from inner scopes in the script scope

src/ast/scopes.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_AST_SCOPES_H_
 #define V8_AST_SCOPES_H_
 
 #include "src/ast/variables.h"
 #include "src/base/hashmap.h"
 #include "src/globals.h"
@@ skipping 181 matching lines @@
   // `try{}catch(e){let e;}`,
   // which is an error even though the two 'e's are declared in different
   // scopes.
   Declaration* CheckLexDeclarationsConflictingWith(
       const ZoneList<const AstRawString*>& names);
 
   // ---------------------------------------------------------------------------
   // Scope-specific info.
 
   // Inform the scope and outer scopes that the corresponding code contains an
-  // eval call.
+  // eval call. We don't record eval calls from innner scopes in the outer most
+  // script scope, as we only see those when parsing eagerly. If we recorded the
+  // calls then, the outer most script scope would look different depending on
+  // whether we parsed eagerly or not which is undesirable.
   void RecordEvalCall() {
     scope_calls_eval_ = true;
     inner_scope_calls_eval_ = true;
     for (Scope* scope = outer_scope(); scope && !scope->is_script_scope();
          scope = scope->outer_scope()) {
       scope->inner_scope_calls_eval_ = true;
     }
   }
 
   // Set the language mode flag (unless disabled by a global flag).
@@ skipping 637 matching lines @@
   void AllocateModuleVariables();
 
  private:
   ModuleDescriptor* module_descriptor_;
 };
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_AST_SCOPES_H_