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

Side by Side Diff: src/compiler.cc

Issue 131383004: [not for landing] Diff between a64 and r19234 for no-a64 files. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 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 289 matching lines...) Expand 10 before | Expand all | Expand 10 after
300 300
301 OptimizedCompileJob::Status OptimizedCompileJob::CreateGraph() { 301 OptimizedCompileJob::Status OptimizedCompileJob::CreateGraph() {
302 ASSERT(isolate()->use_crankshaft()); 302 ASSERT(isolate()->use_crankshaft());
303 ASSERT(info()->IsOptimizing()); 303 ASSERT(info()->IsOptimizing());
304 ASSERT(!info()->IsCompilingForDebugging()); 304 ASSERT(!info()->IsCompilingForDebugging());
305 305
306 // We should never arrive here if there is no code object on the 306 // We should never arrive here if there is no code object on the
307 // shared function object. 307 // shared function object.
308 ASSERT(info()->shared_info()->code()->kind() == Code::FUNCTION); 308 ASSERT(info()->shared_info()->code()->kind() == Code::FUNCTION);
309 309
310 if (FLAG_trace_opt) {
ulan 2014/02/11 17:24:26 Remove before landing?
Rodolph Perfetta 2014/02/11 17:58:29 Yes remove. On 2014/02/11 17:24:26, ulan wrote:
311 // TODO(jbramley): This was added to help analyse the behaviour of
312 // Crankshaft for the A64 Crankshaft port. It should eventually be deleted.
313 Handle<JSFunction> function = info()->closure();
314 PrintF("Attempting optimization of ");
315 function->PrintName();
316 PrintF(" / %" V8PRIxPTR "\n", reinterpret_cast<intptr_t>(*function));
317 }
318
310 // We should never arrive here if optimization has been disabled on the 319 // We should never arrive here if optimization has been disabled on the
311 // shared function info. 320 // shared function info.
312 ASSERT(!info()->shared_info()->optimization_disabled()); 321 ASSERT(!info()->shared_info()->optimization_disabled());
313 322
314 // Fall back to using the full code generator if it's not possible 323 // Fall back to using the full code generator if it's not possible
315 // to use the Hydrogen-based optimizing compiler. We already have 324 // to use the Hydrogen-based optimizing compiler. We already have
316 // generated code for this from the shared function object. 325 // generated code for this from the shared function object.
317 if (FLAG_always_full_compiler) return AbortOptimization(); 326 if (FLAG_always_full_compiler) return AbortOptimization();
318 if (IsDebuggerActive(isolate())) return AbortOptimization(kDebuggerIsActive); 327 if (IsDebuggerActive(isolate())) return AbortOptimization(kDebuggerIsActive);
319 328
(...skipping 971 matching lines...) Expand 10 before | Expand all | Expand 10 after
1291 AllowHandleDereference allow_deref; 1300 AllowHandleDereference allow_deref;
1292 bool tracing_on = info()->IsStub() 1301 bool tracing_on = info()->IsStub()
1293 ? FLAG_trace_hydrogen_stubs 1302 ? FLAG_trace_hydrogen_stubs
1294 : (FLAG_trace_hydrogen && 1303 : (FLAG_trace_hydrogen &&
1295 info()->closure()->PassesFilter(FLAG_trace_hydrogen_filter)); 1304 info()->closure()->PassesFilter(FLAG_trace_hydrogen_filter));
1296 return (tracing_on && 1305 return (tracing_on &&
1297 OS::StrChr(const_cast<char*>(FLAG_trace_phase), name_[0]) != NULL); 1306 OS::StrChr(const_cast<char*>(FLAG_trace_phase), name_[0]) != NULL);
1298 } 1307 }
1299 1308
1300 } } // namespace v8::internal 1309 } } // namespace v8::internal
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698