11313 - XPTCInvoke broken for SUNWspro4.2

Status: VERIFIED FIXED

(Core :: XPCOM, defect, P3)

Description

[tor]

TestXPTCInvoke (and apprunner) don't work on a SUNWspro4.2 build after
the changes to xptcall sparc assembly on 8/4/99.  This is as far as it
gets:

(dbx) run
Running: TestXPTCInvoke
(process id 14127)
calling direct:
        1 + 1 = 2
        1L + 1L = 2
        2 * 2 = 4
        2L * 2L = 4
        1 + 2 + 3 + 4 + 5 + 6 + 7 + 8 + 9 + 10 = 55.000000
        1 + 2 + 3 + 4 + 5 + 6 + 7 + 8 + 9 + 10 = 55.000000f
calling via invoke:
        1 + 1 = 0
        1L + 1L = 0
t@1 (l@1) signal SEGV (no mapping at the fault address) in (unknown) at 0x0
0x00000000:     <bad address 0x0>
Current function is main
  227       if(NS_SUCCEEDED(XPTC_InvokeByIndex(test, 4, 3, var)))
(dbx) where
current thread: t@1
  [1] 0x0(0x27eb8, 0x2, 0x2, 0xffbef510, 0x1, 0xffbef510), at 0xffffffff
  [2] XPTC_InvokeByIndex(0x27eb8, 0x4, 0x3, 0xffbef4f0, 0xfeeb5c58, 0xffffffff),
at 0xff301678
=>[3] main(), line 227 in "TestXPTCInvoke.cpp"

Comment 1

[Chris McAfee]

adding mcafee, scc.

Comment 2

[rogerl (gone)]

Argh, nuts. Did this used to work previously with just the generic build? If so
it implies that the 4.x workshop uses the same vtable layout as GCC. The code I
checked in was developed for the 5.0 workshop. I guess I need to test for a
specific Workshop version in the Makefile.
(Chris, do we have a sparc solaris machine with any Workshop version that
can be used to do a build?)

Comment 3

[tor]

Yes, SUNWspro 4.2 previously worked fine with a generic build.
Guess we just lucked out before...

Comment 4

[rogerl (gone)]

ok, just checked in changes to go back to GCC vtables for SunW builds. Matthias
at Sun says he's figured out how to get the 5.0 workshop to produce 4.2 & GCC
compatible vtables, I'm waiting for him to give me the gory details but this bug
should go away now.

Comment 5

[rubydoo123]

tor -- can you verify this on your Sun? thanks

Comment 6

[tor]

TestXPC, TestXPTCInvoke, and apprunner now run fine with a Solaris/SUNWspro4.2
build (8/15 cvs pull).