[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 ktietz at gcc dot gnu.org 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=gccview=revrev=184526 Log: PR libffi/52221

[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 ktietz at gcc dot gnu.org changed: What|Removed |Added Status|WAITING |RESOLVED

[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 jojelino at gmail dot com 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

[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 jojelino at gmail dot com 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

[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 ktietz at gcc dot gnu.org 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-16 Thread jojelino at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52221 --- Comment #8 from gee jojelino at gmail dot com 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'

[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 ktietz at gcc dot gnu.org 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

[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 jojelino at gmail dot com changed: What|Removed |Added Attachment #26653|0 |1 is obsolete|

[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 jojelino at gmail dot com 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

[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 jojelino at gmail dot com changed: What|Removed |Added Summary|[libffi] r184021 needs to |[libffi] r183675,r184021