Bug 77150 - [JSC] Asm-originated symbols should be marked as hidden
Summary: [JSC] Asm-originated symbols should be marked as hidden
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Simon Hausmann
URL:
Keywords:
Depends on:
Blocks: 72854
  Show dependency treegraph
 
Reported: 2012-01-26 15:06 PST by Hajime Morrita
Modified: 2012-01-27 14:43 PST (History)
3 users (show)

See Also:


Attachments
Patch (2.35 KB, patch)
2012-01-27 03:51 PST, Simon Hausmann
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Hajime Morrita 2012-01-26 15:06:50 PST
This is from Bug 72854.

Following symbols are currently not marked as hidden,
that means these will be visible if we switched off the exp list based symbol management.
We need to mark them hidden.

----
__ZThn16_N3JSC9CodeBlockD0Ev
__ZThn16_N3JSC9CodeBlockD1Ev
_ceilThunk
_expThunk
_floorThunk
_jsRoundThunk
_logThunk

_getHostCallReturnValue
_operationGetByIdBuildList
_operationGetByIdOptimize
_operationGetByIdProtoBuildList
_operationLinkCall
_operationLinkConstruct
_operationPutByIdDirectNonStrictOptimize
_operationPutByIdDirectStrictOptimize
_operationPutByIdNonStrictOptimize
_operationPutByIdStrictOptimize
----
Comment 1 Simon Hausmann 2012-01-27 03:51:36 PST
Created attachment 124285 [details]
Patch
Comment 2 Hajime Morrita 2012-01-27 09:01:29 PST
CC-ing Filip, an expert in this area.
Comment 3 Filip Pizlo 2012-01-27 13:30:47 PST
Comment on attachment 124285 [details]
Patch

I take it there will be additional packages to hide the various other symbols?  R=me.
Comment 4 Hajime Morrita 2012-01-27 13:48:05 PST
(In reply to comment #3)
> (From update of attachment 124285 [details])
> I take it there will be additional packages to hide the various other symbols?  R=me.
Yeah, this is a part of effort to kill JavaScriptCore.exp with function-based annotation  JS_EXPORT_PRIVATE.
Thanks for your quick take, Filip.
Comment 5 Filip Pizlo 2012-01-27 13:48:40 PST
(In reply to comment #3)
> (From update of attachment 124285 [details])
> I take it there will be additional packages to hide the various other symbols?  R=me.

Ooops, sorry I meant to say "patches" not "packages".
Comment 6 WebKit Review Bot 2012-01-27 14:43:49 PST
Comment on attachment 124285 [details]
Patch

Clearing flags on attachment: 124285

Committed r106153: <http://trac.webkit.org/changeset/106153>
Comment 7 WebKit Review Bot 2012-01-27 14:43:54 PST
All reviewed patches have been landed.  Closing bug.