[turbofan] Fix iteration order

src/compiler/escape-analysis.h

 // Copyright 2015 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_COMPILER_ESCAPE_ANALYSIS_H_
 #define V8_COMPILER_ESCAPE_ANALYSIS_H_
 
 #include "src/base/flags.h"
 #include "src/compiler/graph.h"
 
 namespace v8 {
 namespace internal {
 namespace compiler {
 
 // Forward declarations.
 class CommonOperatorBuilder;
 class EscapeAnalysis;
 class VirtualState;
 class VirtualObject;
 
-
 // EscapeStatusAnalysis determines for each allocation whether it escapes.
 class EscapeStatusAnalysis {
  public:
   typedef NodeId Alias;
   ~EscapeStatusAnalysis();
 
   enum Status {
     kUnknown = 0u,
     kTracked = 1u << 0,
     kEscaped = 1u << 1,
     kOnStack = 1u << 2,
     kVisited = 1u << 3,
     // A node is dangling, if it is a load of some kind, and does not have
     // an effect successor.
     kDanglingComputed = 1u << 4,
     kDangling = 1u << 5,
     // A node is is an effect branch point, if it has more than 2 non-dangling
     // effect successors.
     kBranchPointComputed = 1u << 6,
     kBranchPoint = 1u << 7,
+    kInQueue = 1u << 8
   };
-  typedef base::Flags<Status, unsigned char> StatusFlags;
+  typedef base::Flags<Status, uint16_t> StatusFlags;
 
   void RunStatusAnalysis();
 
   bool IsVirtual(Node* node);
   bool IsEscaped(Node* node);
   bool IsAllocation(Node* node);
+
+  bool IsInQueue(NodeId id);
+  void SetInQueue(NodeId id, bool on_stack);
+
   void DebugPrint();
 
   EscapeStatusAnalysis(EscapeAnalysis* object_analysis, Graph* graph,
                        Zone* zone);
   void EnqueueForStatusAnalysis(Node* node);
   bool SetEscaped(Node* node);
   bool IsEffectBranchPoint(Node* node);
   bool IsDanglingEffectNode(Node* node);
   void ResizeStatusVector();
   size_t GetStatusVectorSize();
@@ skipping 35 matching lines @@
   Graph* const graph_;
   Zone* const zone_;
   ZoneVector<StatusFlags> status_;
   Alias next_free_alias_;
   ZoneVector<Node*> status_stack_;
   ZoneVector<Alias> aliases_;
 
   DISALLOW_COPY_AND_ASSIGN(EscapeStatusAnalysis);
 };
 
-
 DEFINE_OPERATORS_FOR_FLAGS(EscapeStatusAnalysis::StatusFlags)
 
-
 // Forward Declaration.
 class MergeCache;
 
-
 // EscapeObjectAnalysis simulates stores to determine values of loads if
 // an object is virtual and eliminated.
 class EscapeAnalysis {
  public:
   using Alias = EscapeStatusAnalysis::Alias;
   EscapeAnalysis(Graph* graph, CommonOperatorBuilder* common, Zone* zone);
   ~EscapeAnalysis();
 
   void Run();
 
@@ skipping 66 matching lines @@
   MergeCache* cache_;
 
   DISALLOW_COPY_AND_ASSIGN(EscapeAnalysis);
 };
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_COMPILER_ESCAPE_ANALYSIS_H_