[Interpreter] Avoid dereferencing handles on BytecodeGenerator for AST operations.

src/globals.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_GLOBALS_H_
 #define V8_GLOBALS_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 797 matching lines @@
 // implemented according to ES6, section 7.1.1.
 enum class ToPrimitiveHint { kDefault, kNumber, kString };
 
 // Defines specifics about arguments object or rest parameter creation.
 enum class CreateArgumentsType : uint8_t {
   kMappedArguments,
   kUnmappedArguments,
   kRestParameter
 };
 
+enum class HandleDereferenceMode {
+  kHandleDereferenceAllowed,

[adamk, 2016/08/05 18:38:10]

These can just be kAllowed and kDisallowed, since you're using an enum class.

[marja, 2016/08/08 08:20:13]

+1

[rmcilroy, 2016/08/08 13:05:56]

Good point! :). Done.

+  kHandleDereferenceDisallowed
+};
+
 inline size_t hash_value(CreateArgumentsType type) {
   return bit_cast<uint8_t>(type);
 }
 
 inline std::ostream& operator<<(std::ostream& os, CreateArgumentsType type) {
   switch (type) {
     case CreateArgumentsType::kMappedArguments:
       return os << "MAPPED_ARGUMENTS";
     case CreateArgumentsType::kUnmappedArguments:
       return os << "UNMAPPED_ARGUMENTS";
@@ skipping 320 matching lines @@
   return static_cast<uint32_t>(bit_cast<uintptr_t>(address) >>
                                kPointerSizeLog2);
 }
 
 }  // namespace internal
 }  // namespace v8
 
 namespace i = v8::internal;
 
 #endif  // V8_GLOBALS_H_