Closed Bug 675867 Opened 13 years ago Closed 13 years ago

Enable Identical Code Folding when supported by the linker

Categories

(Firefox Build System :: General, defect)

All
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla8

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(1 file)

      No description provided.
Assignee: nobody → mh+mozilla
This is quite bothering. The latest released version of gold (2.21.1), which supports ICF, crashes when building our code, but not when building the small test case. I tried to reduce some source code that triggers the crash, but it's still more than a thousand lines of code...
I'm tempted to default to disable icf and enable it with mozconfig when we have a toolchain that supports ICF and doesn't crash, but it means linux distros will have to be aware of the option...

On the other hand, it only seems to crash when targetting ARM, so maybe the risk is pretty limited.
Attachment #550060 - Flags: review?(ted.mielczarek)
Comment on attachment 550060 [details] [diff] [review]
Enable Identical Code Folding when supported by the linker

Review of attachment 550060 [details] [diff] [review]:
-----------------------------------------------------------------

Is there a particular reason you need to sanity-check that the functions actually got folded, and you can't just trust that the linker does the right thing when you pass that option?
Attachment #550060 - Flags: review?(ted.mielczarek) → review+
(In reply to Ted Mielczarek [:ted, :luser] from comment #2)
> Is there a particular reason you need to sanity-check that the functions
> actually got folded, and you can't just trust that the linker does the right
> thing when you pass that option?

Android NDK's ld 2.19 accepts the option but does nothing. It's better to avoid surprises as to why -Wl,--icf is used but has no effect...
http://hg.mozilla.org/mozilla-central/rev/4c3bcc010d85
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: