550405 - [Sun Studio] Mangled name mismatch for LoadInlineStyle and LoadStyleLink (release build only)

Status: RESOLVED FIXED

(Core :: DOM: CSS Object Model, defect)

Description

[Ginn Chen]

ld: fatal: symbol referencing errors. No output written to nsIFileEnumerator
gmake[4]: *** [nsIFileEnumerator] Error 1
Undefined                       first referenced
 symbol                             in file
unsigned mozilla::css::Loader::LoadStyleLink(nsIContent*,nsIURI*,const nsAString_internal&,const nsAString_internal&,int,nsICSSLoaderObserver*,int*) ../../dist/lib/libxul.so
Undefined                       first referenced
 symbol                             in file
unsigned mozilla::css::Loader::LoadInlineStyle(nsIContent*,nsIUnicharInputStream*,unsigned,const nsAString_internal&,const nsAString_internal&,nsICSSLoaderObserver*,int*,int*) ../../dist/lib/libxul.so
unsigned mozilla::css::Loader::LoadStyleLink(nsIContent*,nsIURI*,const nsAString_internal&,const nsAString_internal&,int,nsICSSLoaderObserver*,int*) ../../dist/lib/libxul.so
ld: fatal: symbol referencing errors. No output written to TestRegistrationOrder
unsigned mozilla::css::Loader::LoadInlineStyle(nsIContent*,nsIUnicharInputStream*,unsigned,const nsAString_internal&,const nsAString_internal&,nsICSSLoaderObserver*,int*,int*) ../../dist/lib/libxul.so
ld: fatal: symbol referencing errors. No output written to TestCallTemplates

nm toolkit/library/libxul.so | grep LoadStyleLink
[135060]        |         0|         0|FUNC |GLOB |0    |UNDEF  |__1cHmozillaDcssGLoaderNLoadStyleLink6MpnKnsIContent_pnGnsIURI_rknSnsAString_internal_9AipnUnsICSSLoaderObserver_pi_I_
[3668]  |  13018708|      1401|FUNC |LOCL |2    |13     |__1cHmozillaDcssGLoaderNLoadStyleLink6MpnKnsIContent_pnGnsIURI_rknSnsAString_internal_rk7ipnUnsICSSLoaderObserver_pi_I_

nm toolkit/library/libxul.so | grep LoadStyleLink | c++filt 
[135060]        |         0|         0|FUNC |GLOB |0    |UNDEF  |unsigned mozilla::css::Loader::LoadStyleLink(nsIContent*,nsIURI*,const nsAString_internal&,const nsAString_internal&,int,nsICSSLoaderObserver*,int*)
[3668]  |  13018708|      1401|FUNC |LOCL |2    |13     |unsigned mozilla::css::Loader::LoadStyleLink(nsIContent*,nsIURI*,const nsAString_internal&,const nsAString_internal&,int,nsICSSLoaderObserver*,int*)

nm toolkit/library/libxul.so | grep LoadInlineStyle
[136677]        |         0|         0|FUNC |GLOB |0    |UNDEF  |__1cHmozillaDcssGLoaderPLoadInlineStyle6MpnKnsIContent_pnVnsIUnicharInputStream_IrknSnsAString_internal_9ApnUnsICSSLoaderObserver_pi9D_I_
[21555] |  13017900|       808|FUNC |LOCL |2    |13     |__1cHmozillaDcssGLoaderPLoadInlineStyle6MpnKnsIContent_pnVnsIUnicharInputStream_IrknSnsAString_internal_rk7pnUnsICSSLoaderObserver_pi9F_I_

nm toolkit/library/libxul.so | grep LoadInlineStyle | c++filt 
[136677]        |         0|         0|FUNC |GLOB |0    |UNDEF  |unsigned mozilla::css::Loader::LoadInlineStyle(nsIContent*,nsIUnicharInputStream*,unsigned,const nsAString_internal&,const nsAString_internal&,nsICSSLoaderObserver*,int*,int*)
[21555] |  13017900|       808|FUNC |LOCL |2    |13     |unsigned mozilla::css::Loader::LoadInlineStyle(nsIContent*,nsIUnicharInputStream*,unsigned,const nsAString_internal&,const nsAString_internal&,nsICSSLoaderObserver*,int*,int*)

If I add "-g" for compiling nsCSSLoader.cpp, the problem is gone.
Reproduced with both SS12 and SS12U1.

It might be a bug of compiler, I need to consult compiler team.

Comment 1

[Boris Zbarsky [:bzbarsky]]

I assume the GLOB vs LOCL thing is likely to be relevant, right?

Comment 2

[Ginn Chen]

Since it is UNDEF it has to be GLOB.

The issue here is documented at http://developers.sun.com/sunstudio/documentation/ss12u1/mr/READMEs/c++.html#cNameMangling

Perhaps we use actual types and equivalent typedefs inconsistently in the declaration and definition of the affected functions.

Comment 3

[Boris Zbarsky [:bzbarsky]]

> Perhaps we use actual types and equivalent typedefs inconsistently

Yep, exactly.  nsAString vs nsSubstring.

Comment 4

[Ginn Chen]

Attachment: patch

Thanks for pointing out.

Comment 5

[Zack Weinberg (:zwol)]

FWIW, that's a subset of the patch in bug 549795 ;-)

Comment 6

[Ginn Chen]

http://hg.mozilla.org/mozilla-central/rev/5c9e056e348a