507449 - TM: Trace JSOP_GETXPROP for closures

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Comment 1

[David Mandelin [:dmandelin]]

Attachment: Patch

Comment 2

[Brendan Eich [:brendan]]

Comment on attachment 392388 [details] [diff] [review]
Patch

Could the else after return in TraceRecorder::scopeChainProp be fixed too? Thanks,

/be

Comment 3

[Andreas Gal :gal]

Comment on attachment 392388 [details] [diff] [review]
Patch


>+        }
>+        ABORT_TRACE("can't trace special CallClass setter");
>+        
>+    }

That empty line can go IMO.

Comment 4

[David Mandelin [:dmandelin]]

Pushed to TM as 00fa2213bef6.

Comment 5

[Robert Sayre]

http://hg.mozilla.org/mozilla-central/rev/42338f16ac10