Remove unneeded variable usage analysis.

src/scopeinfo.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 64 matching lines @@
   // sort them by context slot index before adding them to the
   // ScopeInfo list.
   List<Variable*, Allocator> locals(32);  // 32 is a wild guess
   ASSERT(locals.is_empty());
   scope->CollectUsedVariables(&locals);
   locals.Sort(&CompareLocal);
 
   List<Variable*, Allocator> heap_locals(locals.length());
   for (int i = 0; i < locals.length(); i++) {
     Variable* var = locals[i];
-    if (var->var_uses()->is_used()) {
+    if (var->is_used()) {
       Slot* slot = var->slot();
       if (slot != NULL) {
         switch (slot->type()) {
           case Slot::PARAMETER:
             // explicitly added to parameters_ above - ignore
             break;
 
           case Slot::LOCAL:
             ASSERT(stack_slots_.length() == slot->index());
             stack_slots_.Add(var->name());
@@ skipping 27 matching lines @@
   } else {
     ASSERT(heap_locals.length() == 0);
   }
 
   // Add the function context slot, if present.
   // For now, this must happen at the very end because of the
   // ordering of the scope info slots and the respective slot indices.
   if (scope->is_function_scope()) {
     Variable* var = scope->function();
     if (var != NULL &&
-        var->var_uses()->is_used() &&
+        var->is_used() &&
         var->slot()->type() == Slot::CONTEXT) {
       function_name_ = var->name();
       // Note that we must not find the function name in the context slot
       // list - instead it must be handled separately in the
       // Contexts::Lookup() function. Thus record an empty symbol here so we
       // get the correct number of context slots.
       ASSERT(var->slot()->index() - Context::MIN_CONTEXT_SLOTS ==
              context_slots_.length());
       ASSERT(var->slot()->index() - Context::MIN_CONTEXT_SLOTS ==
              context_modes_.length());
@@ skipping 496 matching lines @@
 }
 #endif  // DEBUG
 
 
 // Make sure the classes get instantiated by the template system.
 template class ScopeInfo<FreeStoreAllocationPolicy>;
 template class ScopeInfo<PreallocatedStorage>;
 template class ScopeInfo<ZoneListAllocationPolicy>;
 
 } }  // namespace v8::internal