Undo StompWriteBarrierResize reordering

The recent commit introduced race by changing where StompWriteBarrierResize is called relative to other global state updates. Revert this change to fix the intermittent crashes that we are seeing because of that. We will look at fixing the theoretical problem that this reordering was trying to address in better way separately.

Author: jkotas

src/gc/gc.cpp

@@ -7258,17 +7258,6 @@ int gc_heap::grow_brick_card_tables (uint8_t* start,
             g_card_table = translated_ct;
         }
 
-        // We need to make sure that other threads executing checked write barriers
-        // will see the g_card_table update before g_lowest/highest_address updates.
-        // Otherwise, the checked write barrier may AV accessing the old card table
-        // with address that it does not cover. Write barriers access card table 
-        // without memory barriers for performance reasons, so we need to flush 
-        // the store buffers here.
-        GCToOSInterface::FlushProcessWriteBuffers();
-
-        g_lowest_address = saved_g_lowest_address;
-        VolatileStore(&g_highest_address, saved_g_highest_address);
-
         if (!write_barrier_updated)
         {
             // This passes a bool telling whether we need to switch to the post
@@ -7282,6 +7271,17 @@ int gc_heap::grow_brick_card_tables (uint8_t* start,
             StompWriteBarrierResize(!!IsSuspendEEThread(), la != saved_g_lowest_address);
         }
 
+        // We need to make sure that other threads executing checked write barriers
+        // will see the g_card_table update before g_lowest/highest_address updates.
+        // Otherwise, the checked write barrier may AV accessing the old card table
+        // with address that it does not cover. Write barriers access card table 
+        // without memory barriers for performance reasons, so we need to flush 
+        // the store buffers here.
+        GCToOSInterface::FlushProcessWriteBuffers();
+
+        g_lowest_address = saved_g_lowest_address;
+        VolatileStore(&g_highest_address, saved_g_highest_address);
+
         return 0;
 
 fail: