Opened 7 years ago

Closed 7 years ago

#278 closed defect (worksforme)

gcc 4.7.x inline behaviour change

Reported by: Yuri Dario Owned by:
Priority: normal Milestone: libc-0.6.6
Component: libc Version:
Severity: major Keywords:

Description (last modified by bird)

With newer gcc releases, code inlining is following a different path than before. This leads to many problems in the build of libc but also of other applications. See for details.

At this time, this is fixed using -fgnu89-inline which restores old behaviour.

The right thing seems to use 'extern inline' instead of 'static inline'. But without a proper prototype, gcc will still ignore inlining.

Many headers defines code to be inlined without a prototype.

I'm leaving this ticket open for reference.

Attachments (1)

patch-278b.txt (1.3 KB) - added by Yuri Dario 7 years ago.

Download all attachments as: .zip

Change History (5)

Changed 7 years ago by Yuri Dario

Attachment: patch-278b.txt added

comment:1 Changed 7 years ago by Yuri Dario

Trunk doesn't need this patch. Time to check build flags and discover what is changed...

comment:2 Changed 7 years ago by bird

Please, this isn't a place to educate people about the changes in gcc version changes. The defect completely lacks any real error messages and problem description. It will not be left open for reference, it will instead be close immediately the issues you've experienced (whatever they are) has been fully analyzed and addressed.

comment:3 Changed 7 years ago by bird

Description: modified (diff)

I also fail to see the difference between inline and __inline__ while in libc sources. You better look at the preprocessor output for those files with your compiler and see what really compiled, my guess would be that inline is defined to an empty expression for some stupid reason.

There should also be no need for prototyping inlined functions. If you want to start doing that, I require a really good and valid reason for it.

comment:4 Changed 7 years ago by bird

Resolution: worksforme
Status: newclosed
Note: See TracTickets for help on using tickets.