This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Fix PR libffi/60073


> Eric, sorry to rain on your parade, but you're making the same mistake
> I did at first when adding powerpc64le support to libffi.  libffi is
> built as a shared library.  You can't add a field to ffi_cif like this
> and uncondionally write to it:  An application linked against an older
> version of libffi will only allocate the old size ffi_cif.  Your new
> shared library will trash some random location in the old user app..

OK, I didn't realize that libffi was built as a shared library.  That seems a 
little strange and inconvenient for a glue library.

> I worked around this problem on powerpc by defining new enum ffi_abi
> values so that you can recognize an old app.

I see, thanks for the heads up and the hint!

-- 
Eric Botcazou


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]