694119 - CfgLister case LIR_jtbl asserts.

Status: RESOLVED FIXED

(Core Graveyard :: Nanojit, defect)

Description

[Edwin Smith]

Attachment: Obvious fix, plus a bit of whitespace cleanup.

It is calling ins->getTarget() instead of ins->getTarget(i).

Comment 1

[Rick Reitmaier]

Comment on attachment 566618 [details] [diff] [review]
Obvious fix, plus a bit of whitespace cleanup.

r+.

BTW, '[diff] git=true' in your .hgrc file will get rid of the unhelpful file names in bugzilla diff display, assuming this was an hg derived patch.  See 'hg help diffs'

Comment 2

[Edwin Smith]

(In reply to Rick Reitmaier from comment #1)
> Comment on attachment 566618 [details] [diff] [review] [diff] [details] [review]
> Obvious fix, plus a bit of whitespace cleanup.
> 
> r+.
> 
> BTW, '[diff] git=true' in your .hgrc file will get rid of the unhelpful file
> names in bugzilla diff display, assuming this was an hg derived patch.  See
> 'hg help diffs'

yeah, willfix.  need to sync my .hgrc across a couple machines.

Comment 3

[Tamarin Bot]

changeset: 6639:0a6a685cecbc
user:      Edwin Smith <edwsmith>
summary:   Bug 694119 - "CfgLister case LIR_jtbl asserts." [r=rreitmai]

http://hg.mozilla.org/tamarin-redux/rev/0a6a685cecbc