* Antti P Miettinen ([EMAIL PROTECTED]) wrote: > > > Seems that the inputs for the inline asm in XPTC_InvokeByIndex() in > > > xpcom/reflect/xptcall/src/md/unix/xptcinvoke_arm.cpp need an > > > additional memory input in order to prevent gcc from optimizing away > > > the initialization of the structure used by the asm. > > > > Is this a gcc 4 issue? Any idea why this is necessary? > > I suppose one could call it a gcc 4 issue if earlier gcc versions were > unable to optimize away the dead struct init. As far as I understand > the asm contraints, not specifying the structure as a mem input for > the asm, makes the struct init dead code. So the mem input should be > specified anyway - the code just was "lucky" with earlier gcc > versions. But I'm not an expert in gcc asm.. could be something more > subtle..
Ok, fix is in my tree and will be in the next release. Do you mind reporting this upstream? (bugzilla.mozilla.org) I'll do it if you don't want to, but since you provided the patch, it might be better to communicate directly. -- Eric Dorland <[EMAIL PROTECTED]> ICQ: #61138586, Jabber: [EMAIL PROTECTED] 1024D/16D970C6 097C 4861 9934 27A0 8E1C 2B0A 61E9 8ECF 16D9 70C6 -----BEGIN GEEK CODE BLOCK----- Version: 3.12 GCS d- s++: a-- C+++ UL+++ P++ L++ E++ W++ N+ o K- w+ O? M++ V-- PS+ PE Y+ PGP++ t++ 5++ X+ R tv++ b+++ DI+ D+ G e h! r- y+ ------END GEEK CODE BLOCK------
signature.asc
Description: Digital signature
