gecko-dev/js
Boris Zbarsky 84e60db6d8 Bug 1591481 part 2. Don't force copying when external strings are passed to Web IDL bindings. r=mccr8
The codesize impact is approximately 0 (actually positive by a a few KB),
because AssignJSString is not inlined.

If we were to force-inline AssignJSString, that would actually _save_ ~115KB
based on bloaty measurements for linux64 shippable opt builds (so with LTO),
because then ConvertJSValueToString stops being inlined in bindings.
Given that ConvertJSValueToString pretty much always calls AssignJSString, so
there is no slowdown from changing which of them is inlined, this is _really_
tempting to do, but I don't want to mess with compiler inlining heuristics...
Inlining both AssignJSString and ConvertJSValueToString increases codesize by
~2MB, so that's definitely _not_ desirable.

Differential Revision: https://phabricator.services.mozilla.com/D50627

--HG--
extra : moz-landing-system : lando
2019-10-30 15:52:22 +00:00
..
ductwork/debugger
examples
ipc
public Backed out 7 changesets (bug 1582348) for spidermonkey bustage at WritableStream.cpp on a CLOSED TREE. 2019-10-30 07:46:31 +02:00
rust Bug 1448324 - Fix link in js/rust readme. r=jdm 2019-10-18 14:30:52 +00:00
src Bug 1577161 - Limit length of filename saved for profiler. r=jandem 2019-10-30 12:51:57 +00:00
xpconnect Bug 1591481 part 2. Don't force copying when external strings are passed to Web IDL bindings. r=mccr8 2019-10-30 15:52:22 +00:00
app.mozbuild
ffi.configure
moz.build
moz.configure
sub.configure