Fix mjsunit/compiler/concurrent-invalidate-transition-map.js test.

Fix mjsunit/compiler/concurrent-invalidate-transition-map.js test.

BUG=v8:3156
LOG=N
R=yangguo@chromium.org

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

[ulan, 2014-02-25 17:44:28]

PTAL

https://codereview.chromium.org/180053003/diff/1/test/mjsunit/compiler/concur...
File test/mjsunit/compiler/concurrent-invalidate-transition-map.js (right):

https://codereview.chromium.org/180053003/diff/1/test/mjsunit/compiler/concur...
test/mjsunit/compiler/concurrent-invalidate-transition-map.js:47:
add_field(obj);
If GC happens between the calls and there is no live instance of new_object then
then transition in add_field will be collected, so on the next call to add_field
the obj.c IC will not turn MONOMORPHIC.

[Yang, 2014-02-25 17:46:21]

On 2014/02/25 17:44:28, ulan wrote:
> PTAL
> 
>
https://codereview.chromium.org/180053003/diff/1/test/mjsunit/compiler/concur...
> File test/mjsunit/compiler/concurrent-invalidate-transition-map.js (right):
> 
>
https://codereview.chromium.org/180053003/diff/1/test/mjsunit/compiler/concur...
> test/mjsunit/compiler/concurrent-invalidate-transition-map.js:47:
> add_field(obj);
> If GC happens between the calls and there is no live instance of new_object
then
> then transition in add_field will be collected, so on the next call to
add_field
> the obj.c IC will not turn MONOMORPHIC.

LGTM.

[Yang, 2014-03-13 11:57:38]

On 2014/02/25 17:46:21, Yang wrote:
> On 2014/02/25 17:44:28, ulan wrote:
> > PTAL
> > 
> >
>
https://codereview.chromium.org/180053003/diff/1/test/mjsunit/compiler/concur...
> > File test/mjsunit/compiler/concurrent-invalidate-transition-map.js (right):
> > 
> >
>
https://codereview.chromium.org/180053003/diff/1/test/mjsunit/compiler/concur...
> > test/mjsunit/compiler/concurrent-invalidate-transition-map.js:47:
> > add_field(obj);
> > If GC happens between the calls and there is no live instance of new_object
> then
> > then transition in add_field will be collected, so on the next call to
> add_field
> > the obj.c IC will not turn MONOMORPHIC.
> 
> LGTM.

Is this no longer relevant?

[Yang, 2014-03-17 16:04:08]

On 2014/03/13 11:57:38, Yang wrote:
> On 2014/02/25 17:46:21, Yang wrote:
> > On 2014/02/25 17:44:28, ulan wrote:
> > > PTAL
> > > 
> > >
> >
>
https://codereview.chromium.org/180053003/diff/1/test/mjsunit/compiler/concur...
> > > File test/mjsunit/compiler/concurrent-invalidate-transition-map.js
(right):
> > > 
> > >
> >
>
https://codereview.chromium.org/180053003/diff/1/test/mjsunit/compiler/concur...
> > > test/mjsunit/compiler/concurrent-invalidate-transition-map.js:47:
> > > add_field(obj);
> > > If GC happens between the calls and there is no live instance of
new_object
> > then
> > > then transition in add_field will be collected, so on the next call to
> > add_field
> > > the obj.c IC will not turn MONOMORPHIC.
> > 
> > LGTM.
> 
> Is this no longer relevant?

dumdidum :)

[ulan, 2014-03-17 16:12:02]

On 2014/03/17 16:04:08, Yang wrote:
> On 2014/03/13 11:57:38, Yang wrote:
> > On 2014/02/25 17:46:21, Yang wrote:
> > > On 2014/02/25 17:44:28, ulan wrote:
> > > > PTAL
> > > > 
> > > >
> > >
> >
>
https://codereview.chromium.org/180053003/diff/1/test/mjsunit/compiler/concur...
> > > > File test/mjsunit/compiler/concurrent-invalidate-transition-map.js
> (right):
> > > > 
> > > >
> > >
> >
>
https://codereview.chromium.org/180053003/diff/1/test/mjsunit/compiler/concur...
> > > > test/mjsunit/compiler/concurrent-invalidate-transition-map.js:47:
> > > > add_field(obj);
> > > > If GC happens between the calls and there is no live instance of
> new_object
> > > then
> > > > then transition in add_field will be collected, so on the next call to
> > > add_field
> > > > the obj.c IC will not turn MONOMORPHIC.
> > > 
> > > LGTM.
> > 
> > Is this no longer relevant?
> 
> dumdidum :)

Fixed test, landing.

[ulan, 2014-03-17 17:00:54]

Committed patchset #2 manually as r20009 (presubmit successful).