1121363 - Remove more straggling PJS bits

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Lars T Hansen [:lth]]

By an oversight, this function was not removed earlier.  grep finds these instances:

./src/builtin/TestingFunctions.cpp:1226:js::testingFunc_inParallelSection(JSContext *cx, unsigned argc, jsval *vp)
./src/builtin/TestingFunctions.cpp:2459:    JS_FN_HELP("inParallelSection", testingFunc_inParallelSection, 0, 0,
./src/builtin/TestingFunctions.cpp:2460:"inParallelSection()",
./src/builtin/TestingFunctions.h:18:testingFunc_inParallelSection(JSContext *cx, unsigned argc, Value *vp);

Comment 1

[Lars T Hansen [:lth]]

By an oversight, lots of other little bits (and some larger bits) were also not removed earlier.  I've filed bugs for some of them (bug 1121433, bug 1121434, bug 1121435) but the easy fixes are handled here.

Comment 2

[Lars T Hansen [:lth]]

Attachment: Remove yet more PJS code

Hannes, can you check that my changes to TraceLogger don't do anything bad?

Comment 3

[Hannes Verschore [:h4writer]]

Comment on attachment 8548870 [details] [diff] [review]
Remove yet more PJS code

Review of attachment 8548870 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good wrt Tracelogger. Seems like TraceLogger_ParallelSafetyAnalysis was never used or did you already remove that?

Comment 4

[Lars T Hansen [:lth]]

(In reply to Hannes Verschore [:h4writer] from comment #3)
> Comment on attachment 8548870 [details] [diff] [review]
> Remove yet more PJS code
> 
> Review of attachment 8548870 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Looks good wrt Tracelogger. Seems like TraceLogger_ParallelSafetyAnalysis
> was never used or did you already remove that?

Any use would have been removed by an earlier patch, when the bulk of the JIT was cleaned out.

Comment 5

[Shu-yu Guo [:shu]]

Comment on attachment 8548870 [details] [diff] [review]
Remove yet more PJS code

Review of attachment 8548870 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/jsobj.cpp
@@ -3286,5 @@
> -    if (name->isIndex(&index))
> -        return GetElementPure(cx, obj, index, vp);
> -
> -    return GetPropertyPure(cx, obj, NameToId(name->asPropertyName()), vp);
> -}

Hm, seeing how LookupPropertyPure is independently useful, I wonder if these pure variants would also be independently useful. I suppose it doesn't hurt to remove now and add them back later should new users arise, rather than let them rot in the codebase.

Comment 6

[Shu-yu Guo [:shu]]

Attachment: Simplify VMFunction context matching.

Might as well just post it in this bug. Turned out there wasn't really anything
big I needed to do with VMFunctions. Sorry for not r+ing your deleting this a
while back, Lars.

Comment 7

[Lars T Hansen [:lth]]

Comment on attachment 8549272 [details] [diff] [review]
Simplify VMFunction context matching.

Review of attachment 8549272 [details] [diff] [review]:
-----------------------------------------------------------------

I'll land this this morning when I land the other one.

Comment 8

[Lars T Hansen [:lth]]

Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=0962bd11452a

Comment 9

[Lars T Hansen [:lth]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2b9b821f28a7
https://hg.mozilla.org/integration/mozilla-inbound/rev/97b9f0cf33bf

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/2b9b821f28a7
https://hg.mozilla.org/mozilla-central/rev/97b9f0cf33bf