X87: Disable the test case for X87 since f3338dd3b01c

X87: Disable the test case for X87 since f3338dd3b01c

  The overwrite-builtins test cases will throw one expected exception
  and the exception message should include the source position in the JS
  file. But crankshaft compiler does not generate the source position for
  it when creating the flow graph by default. The source position information
  is always zero. So it failed when comparing with the reference file.

  If we use crankshaft compiler on IA32 platform to run this test case, it
  has the same failure.

BUG=

Committed: https://crrev.com/31a3d5fcd2157bdc002ec7f581905e659eac707b
Cr-Commit-Position: refs/heads/master@{#27807}

[chunyang.dai, 2015-04-13 08:56:01]

Patchset #1 (id:1) has been deleted

[chunyang.dai, 2015-04-13 09:03:23]

chunyang.dai@intel.com changed reviewers:
+ danno@chromium.org, mstarzinger@chromium.org, weiliang.lin@intel.com

[chunyang.dai, 2015-04-13 09:03:23]

PTAL.

[Michael Starzinger, 2015-04-13 20:03:21]

I have the strong suspicion that this will be fixed by one of my pending CLs. If
that's the case then I would suggest simply waiting for that CL to land. WDYT?

See this CL: https://codereview.chromium.org/1058553004/

[Michael Starzinger, 2015-04-13 20:07:01]

Ah, I misread the test case, it's not actually performing a call, it's loading a
property. Maeh, sorry for the noise, I will try to actually fix the issue
tomorrow nonetheless.

[Weiliang, 2015-04-14 01:32:55]

The CQ bit was checked by weiliang.lin@intel.com

[Weiliang, 2015-04-14 01:32:55]

lgtm

[commit-bot: I haz the power, 2015-04-14 01:33:10]

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

[commit-bot: I haz the power, 2015-04-14 01:44:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-14 01:44:46]

Try jobs failed on following builders:
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/1964)

[Weiliang, 2015-04-14 01:47:42]

On 2015/04/14 01:44:46, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   v8_presubmit on tryserver.v8 (JOB_FAILED,
> http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/1964)

Sorry, 

Seems we need V8 team's LGTM

[Michael Starzinger, 2015-04-14 07:55:20]

LGTM. I still think we should add the missing source position.

[chunyang.dai, 2015-04-14 08:03:46]

Hi, Michael.

Thank you very much to review it. I will land it first.

If IA32 platform add the code to add the missing source position for loading a
property, I will port
them to X87.
thanks.

[chunyang.dai, 2015-04-14 08:03:57]

The CQ bit was checked by chunyang.dai@intel.com

[commit-bot: I haz the power, 2015-04-14 08:04:10]

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

[commit-bot: I haz the power, 2015-04-14 08:05:40]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2015-04-14 08:05:54]

Patchset 1 (id:??) landed as
https://crrev.com/31a3d5fcd2157bdc002ec7f581905e659eac707b
Cr-Commit-Position: refs/heads/master@{#27807}

[danno, 2015-04-14 08:24:52]

lgtm