On Thu, Nov 13, 2014 at 8:59 PM, Jeff Law <l...@redhat.com> wrote: > On 11/13/14 21:46, H.J. Lu wrote: > >> >> Hi Jeff, >> >> I believe the patch is correct. Not all glibcs support IFUNC. It doesn't >> mean >> we should disable IFUNC for all glibcs. By default, we do a configure >> time >> check for IFUNC. We DO want to use configure time check for IFUNC for >> Android NDK. >> I > > I'm sorry, I'm objecting to this patch. It is wrong for GCC to produce code > that is incorrect solely for the benefit of optimizing for a new release of > a library. > > The right way is to trigger the new code on some configure time option or a > new configuration itself. > > Whether or not GCC is doing this right for glibc is irrelevant, the patch is > _wrong_. >
Hi Jeff, I have checked in my patch: https://gcc.gnu.org/ml/gcc-cvs/2014-11/msg00594.html so that IFUNC is disabled by default for both Android and uclibc. Is Alexander's patch OK for trunk? Thanks. -- H.J.