Do not left-trim arrays when concurrent sweeping is active.

src/spaces.h

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 457 matching lines @@
   intptr_t GetFlags() { return flags_; }
 
 
   // PARALLEL_SWEEPING_PENDING - This page is ready for parallel sweeping.
   // PARALLEL_SWEEPING_IN_PROGRESS - This page is currently swept or was
   // swept by a sweeper thread.
   // PARALLEL_SWEEPING_DONE - The page state when sweeping is complete or
   // sweeping must not be performed on that page.
   enum ParallelSweepingState {
     PARALLEL_SWEEPING_DONE,
+    PARALLEL_SWEEPING_FINALIZE,

[Michael Starzinger, 2014/03/24 12:21:35]

Please add a comment explaining the semantics of this new state above. Also
please leave a comment that the order of the constants is important because we
use them for range checks.

[Hannes Payer (out of office), 2014/03/25 11:41:08]

Done.

     PARALLEL_SWEEPING_IN_PROGRESS,
     PARALLEL_SWEEPING_PENDING
   };
 
   ParallelSweepingState parallel_sweeping() {
     return static_cast<ParallelSweepingState>(
         NoBarrier_Load(&parallel_sweeping_));
   }
 
   void set_parallel_sweeping(ParallelSweepingState state) {
@@ skipping 2490 matching lines @@
   }
   // Must be small, since an iteration is used for lookup.
   static const int kMaxComments = 64;
 };
 #endif
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_SPACES_H_