Request for review (S) 6588413: Use -fvisibility=hidden for gcc compiles (original) (raw)
Dmitry Samersoff Dmitry.Samersoff at oracle.com
Fri Jan 21 09:47:25 PST 2011
- Previous message: Request for review (S) 6588413: Use -fvisibility=hidden for gcc compiles
- Next message: Request for review (S) 6588413: Use -fvisibility=hidden for gcc compiles
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Coleen,
Hotspot uses a mapfile to hide unused symbols - could we just change the mapfile?
JNICALL is empty in both parts of #ifdef.
-Dmitry
On 2011-01-21 19:57, Coleen Phillimore wrote:
Summary: Add option for gcc 4 and above, define JNIEXPORT and JNIIMPORT to visibility=default, add for jiosnprintf and others since -fvisibility=hidden overrides --version-script definitions.
Performance increases measured by Volker and Tom; 5% (JBB2005) and 2% (JVM98) on Linux/IA64 and 1,5% (JBB2005) and 0,5% (JVM98) on Linux/PPC64
on intel hardware: specjvm98 15 314.61 0.00 0.33 0.006 Yes javac 15 248.88 0.01 1.72 0.000 Yes jack 15 294.44 0.01 1.38 0.000 Yes open webrev at http://cr.openjdk.java.net/~coleenp/6588413/ bug link at http://bugs.sun.com/viewbug.do?bugid=6588413 Tested with jmap for any serviceability impact, and tested hserrfile decoding still works. Thanks, Coleen
-- Dmitry Samersoff Java Hotspot development team, SPB04
- There will come soft rains ...
- Previous message: Request for review (S) 6588413: Use -fvisibility=hidden for gcc compiles
- Next message: Request for review (S) 6588413: Use -fvisibility=hidden for gcc compiles
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]