[Mono-bugs] [Bug 386415] [PATCH] Implementation of mono_method_get_unmanaged_thunk

[bugzilla_noreply at novell.com]

https://bugzilla.novell.com/show_bug.cgi?id=386415

User robertj at gmx.net added comment
https://bugzilla.novell.com/show_bug.cgi?id=386415#c25





--- Comment #25 from Robert Jordan <robertj at gmx.net>  2008-05-05 15:12:54 MST ---
Done that. Now it fails on amd64 due to this optimization for non-pinvoke sigs:

        if (!sig->pinvoke || (size == 0) || (size > 16)) {
                /* Allways pass in memory */
                ainfo->offset = *stack_size;
                *stack_size += ALIGN_TO (size, 8);
                ainfo->storage = ArgOnStack;

                return;
        }

I think we should stick with pinvoke = 1 :-) Otherwise I'd have to review all
mini-<arch>.c to see how far they are ABI compatible for pinvoke = 0.

If none of both patches comes into question, I'll go for #22.


-- 
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.