Update test case TurboAsmDisablesNeuter for "Disable neutering on embedded arraybuffers"

update test case for "Disable neutering on embedded arraybuffers"

  This test case ( added by git: 4c7effe56e8dbe8e2df51bab967c8e1d251f3a66 ).
  should has different implementation for Turbofan and crankshaft.

BUG=

[chunyang.dai, 2014-12-08 10:11:40]

chunyang.dai@intel.com changed reviewers:
+ dcarney@chromium.org

[chunyang.dai, 2014-12-08 10:12:15]

hello, Dcarney.
PTAL.
thanks.

[dcarney, 2014-12-08 10:21:59]

lgtm with the requested change

https://codereview.chromium.org/774163005/diff/1/test/cctest/test-api.cc
File test/cctest/test-api.cc (right):

https://codereview.chromium.org/774163005/diff/1/test/cctest/test-api.cc#newc...
test/cctest/test-api.cc:24492: bool should_be_neuterable = !i::FLAG_turbo_asm;
please just make this

#if V8_TURBOFAN_TARGET
bool should_be_neuterable = !i::FLAG_turbo_asm;
#else 
bool should_be_neuterable = false;
#endif

[chunyang.dai, 2014-12-08 14:38:05]

Hi, Dcarney.
I updated it as the following:

#if V8_TURBOFAN_TARGET
bool should_be_neuterable = !i::FLAG_turbo_asm;
#else 
bool should_be_neuterable = true;
#endif.

For turbofan unspported platform, should_be_neuteralbe is true.

Do I need to wait for your another LGTM ?

thanks.

[dcarney, 2014-12-08 14:46:23]

On 2014/12/08 14:38:05, chunyang.dai wrote:
> Hi, Dcarney.
> I updated it as the following:
> 
> #if V8_TURBOFAN_TARGET
> bool should_be_neuterable = !i::FLAG_turbo_asm;
> #else 
> bool should_be_neuterable = true;
> #endif.
> 
> For turbofan unspported platform, should_be_neuteralbe is true.
> 
> Do I need to wait for your another LGTM ?
> 
> thanks.

one lgtm is enough

[dcarney, 2014-12-08 14:46:28]

The CQ bit was checked by dcarney@chromium.org

[commit-bot: I haz the power, 2014-12-08 14:46:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/774163005/20001

[commit-bot: I haz the power, 2014-12-08 15:14:54]

Committed patchset #2 (id:20001)