[Bug libffi/52221] [libffi] r183675,r184021 needs to be fixed.

2012-02-23 Thread ktietz at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52221 Kai Tietz changed: What|Removed |Added Status|WAITING |RESOLVED Resolution|

[Bug libffi/52221] [libffi] r183675,r184021 needs to be fixed.

2012-02-23 Thread ktietz at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52221 --- Comment #15 from Kai Tietz 2012-02-23 21:02:32 UTC --- Author: ktietz Date: Thu Feb 23 21:02:27 2012 New Revision: 184526 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=184526 Log: PR libffi/52221 * src/x86/ffi.c (ffi_

[Bug libffi/52221] [libffi] r183675,r184021 needs to be fixed.

2012-02-22 Thread jojelino at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52221 --- Comment #14 from gee 2012-02-22 18:13:35 UTC --- (In reply to comment #13) > Yes, this looks better IMHO. Btw any differences about libjava testsuite run > about this altered libffi patch? WARNING: Couldn't find the global config file. Test

[Bug libffi/52221] [libffi] r183675,r184021 needs to be fixed.

2012-02-17 Thread ktietz at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52221 --- Comment #13 from Kai Tietz 2012-02-17 09:03:02 UTC --- Yes, this looks better IMHO. Btw any differences about libjava testsuite run about this altered libffi patch?

[Bug libffi/52221] [libffi] r183675,r184021 needs to be fixed.

2012-02-17 Thread jojelino at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52221 --- Comment #12 from gee 2012-02-17 08:38:01 UTC --- Created attachment 26692 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26692 proposed patch without trampoline diff (In reply to comment #10) > I see here three issues about this patch. >

[Bug libffi/52221] [libffi] r183675,r184021 needs to be fixed.

2012-02-16 Thread jojelino at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52221 --- Comment #11 from gee 2012-02-16 23:19:45 UTC --- (In reply to comment #8) > Created attachment 26685 [details] > proposed patch > > with this patch, ExtraClassLoader testcase passed with three `true' > testing for whole libjava testsuite. Ru

[Bug libffi/52221] [libffi] r183675,r184021 needs to be fixed.

2012-02-16 Thread jojelino at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52221 gee changed: What|Removed |Added Attachment #26653|0 |1 is obsolete||

[Bug libffi/52221] [libffi] r183675,r184021 needs to be fixed.

2012-02-16 Thread ktietz at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52221 --- Comment #10 from Kai Tietz 2012-02-16 20:37:28 UTC --- I see here three issues about this patch. First, you don't treat stack-alignment. Second, why you emulate here stack-argument cleanup for ret __size-4? Why not stick to already present

[Bug libffi/52221] [libffi] r183675,r184021 needs to be fixed.

2012-02-16 Thread jojelino at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52221 --- Comment #8 from gee 2012-02-16 20:12:21 UTC --- Created attachment 26685 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26685 proposed patch with this patch, ExtraClassLoader testcase passed with three `true' testing for whole libjava te

[Bug libffi/52221] [libffi] r183675,r184021 needs to be fixed.

2012-02-15 Thread jojelino at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52221 gee changed: What|Removed |Added Summary|[libffi] r184021 needs to |[libffi] r183675,r184021 |be fixe