Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(479)

Side by Side Diff: src/v8.cc

Issue 574005: Fix issue 597: builtins and stubs are missing in profiler log when using snapshots. (Closed)
Patch Set: Created 10 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2006-2009 the V8 project authors. All rights reserved. 1 // Copyright 2006-2009 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
107 // stack guard. 107 // stack guard.
108 Heap::SetStackLimits(); 108 Heap::SetStackLimits();
109 109
110 // Setup the CPU support. Must be done after heap setup and after 110 // Setup the CPU support. Must be done after heap setup and after
111 // any deserialization because we have to have the initial heap 111 // any deserialization because we have to have the initial heap
112 // objects in place for creating the code object used for probing. 112 // objects in place for creating the code object used for probing.
113 CPU::Setup(); 113 CPU::Setup();
114 114
115 OProfileAgent::Initialize(); 115 OProfileAgent::Initialize();
116 116
117 if (FLAG_log_code) { 117 // If we are deserializing, log non-function code objects and compiled
118 // functions found in the snapshot.
119 if (des != NULL && FLAG_log_code) {
118 HandleScope scope; 120 HandleScope scope;
121 LOG(LogCodeObjects());
119 LOG(LogCompiledFunctions()); 122 LOG(LogCompiledFunctions());
120 } 123 }
121 124
122 return true; 125 return true;
123 } 126 }
124 127
125 128
126 void V8::SetFatalError() { 129 void V8::SetFatalError() {
127 is_running_ = false; 130 is_running_ = false;
128 has_fatal_error_ = true; 131 has_fatal_error_ = true;
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
184 187
185 Smi* V8::RandomPositiveSmi() { 188 Smi* V8::RandomPositiveSmi() {
186 uint32_t random = Random(); 189 uint32_t random = Random();
187 ASSERT(static_cast<uint32_t>(Smi::kMaxValue) >= kRandomPositiveSmiMax); 190 ASSERT(static_cast<uint32_t>(Smi::kMaxValue) >= kRandomPositiveSmiMax);
188 // kRandomPositiveSmiMax must match the value being divided 191 // kRandomPositiveSmiMax must match the value being divided
189 // by in math.js. 192 // by in math.js.
190 return Smi::FromInt(random & kRandomPositiveSmiMax); 193 return Smi::FromInt(random & kRandomPositiveSmiMax);
191 } 194 }
192 195
193 } } // namespace v8::internal 196 } } // namespace v8::internal
OLDNEW
« src/log.cc ('K') | « src/log.cc ('k') | tools/tickprocessor.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698