Fix possible evaluation order problems.

src/bootstrapper.cc

 // Copyright 2006-2008 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 1395 matching lines @@
 void Genesis::InstallJSFunctionResultCaches() {
   const int kNumberOfCaches = 0 +
 #define F(size, func) + 1
     JSFUNCTION_RESULT_CACHE_LIST(F)
 #undef F
   ;
 
   Handle<FixedArray> caches = Factory::NewFixedArray(kNumberOfCaches, TENURED);
 
   int index = 0;
-#define F(size, func) caches->set(index++, CreateCache(size, func));
+
+#define F(size, func) do {                         \
+      FixedArray* cache = CreateCache(size, func); \
+      caches->set(index++, cache);                 \
+  } while (false);

[Erik Corry, 2010/09/23 07:53:28]

You need to lose the semicolon here.

+
     JSFUNCTION_RESULT_CACHE_LIST(F)
+
 #undef F
 
   global_context()->set_jsfunction_result_caches(*caches);
 }
 
 
 void Genesis::InitializeNormalizedMapCaches() {
   Handle<FixedArray> array(
       Factory::NewFixedArray(NormalizedMapCache::kEntries, TENURED));
   global_context()->set_normalized_map_cache(NormalizedMapCache::cast(*array));
@@ skipping 421 matching lines @@
 }
 
 
 // Restore statics that are thread local.
 char* BootstrapperActive::RestoreState(char* from) {
   nesting_ = *reinterpret_cast<int*>(from);
   return from + sizeof(nesting_);
 }
 
 } }  // namespace v8::internal