Fix profiler for TurboFan by reducing duplication.

Fix profiler for TurboFan by reducing duplication.

R=titzer@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=24115

[Michael Starzinger, 2014-09-22 11:04:16]

https://codereview.chromium.org/592703002/diff/1/src/compiler.cc
File src/compiler.cc (right):

https://codereview.chromium.org/592703002/diff/1/src/compiler.cc#newcode704
src/compiler.cc:704: MUST_USE_RESULT static MaybeHandle<Code>
GetCodeFromOptimizedCodeMap(
Note that this entire chunk was just moved, no other changes (except
clang-format) were done here.

[titzer, 2014-09-22 11:27:40]

lgtm with comment

https://codereview.chromium.org/592703002/diff/1/src/compiler.cc
File src/compiler.cc (right):

https://codereview.chromium.org/592703002/diff/1/src/compiler.cc#newcode396
src/compiler.cc:396: (FLAG_turbo_asm && info()->shared_info()->asm_function()))
{
turbo_asm check first will avoid looking at the name and potentially doing the
string matching work.

[Michael Starzinger, 2014-09-22 11:35:37]

Addressed comments. Landing.

https://codereview.chromium.org/592703002/diff/1/src/compiler.cc
File src/compiler.cc (right):

https://codereview.chromium.org/592703002/diff/1/src/compiler.cc#newcode396
src/compiler.cc:396: (FLAG_turbo_asm && info()->shared_info()->asm_function()))
{
On 2014/09/22 11:27:40, titzer wrote:
> turbo_asm check first will avoid looking at the name and potentially doing the
> string matching work.

Done.

[Michael Starzinger, 2014-09-22 12:32:58]

Committed patchset #2 (id:20001) manually as 24115 (presubmit successful).