Closed Bug 1427316 Opened 7 years ago Closed 7 years ago

Use binutils from gcc build when building gcc

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set
normal

Tracking

(firefox59 fixed)

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(2 files)

No description provided.
The differences as per the second patch: linux32: https://public-artifacts.taskcluster.net/LwRx7H0jQ5yzB2cakHgbPg/0/public/diff.html linux64: https://public-artifacts.taskcluster.net/IAut2huJRFmmSJbZ70RKUw/0/public/diff.html Now, for an explanation of the differences (since there are differences, and you can understand why I wanted to reduce them by explicitly disabling .init_array/.fini_array): - Differences in Build ID in notes sections and .gnu_debuglink are due to the fact that the binaries vary (note that debug sections are included in the hashing done for both, so changes in debug info can also trigger that. Changes in debug info can be change in order, change in file paths, change in accuracy, etc. ; we've never concerned ourselves much about debug info changes) - Differences in .dynstr sections are unclear as to why they happen, but all that has no real impact. The contents of .dynstr are used for the .dynsym list, and while that one actually is different because of the offset changes in .dynstr, the list of symbols is actually the same, in the same order. - Inversion of 2 bytes in lock cmpxchg and lock xadd instructions. AIUI, those are actually 2 independent prefix opcodes: F0 is for LOCK, and 66 is an operand-size override prefix, and it doesn't matter in which order they appear. - Replacement of 66 66 66 66 66 2e 0f 1f 84 00 00 00 00 00 with 0f 1f 40 00 66 2e 0f 1f 84 00 00 00 00 00. Those appear to be very large nops to fill space between functions.
Blocks: 1427339
Attachment #8939054 - Flags: review+
Attachment #8939054 - Flags: review?(core-build-config-reviews)
Comment on attachment 8939055 [details] Bug 1427316 - Check differences between builds with and without the changes. https://reviewboard.mozilla.org/r/209474/#review215216 I think it would be worthwhile to have a task that ran diffoscope between the just-produced build and the previous build (from DAG traversal). But this is obviously scope bloat.
(In reply to Gregory Szorc [:gps] from comment #5) > Comment on attachment 8939055 [details] > Bug 1427316 - Check differences between builds with and without the changes. > > https://reviewboard.mozilla.org/r/209474/#review215216 > > I think it would be worthwhile to have a task that ran diffoscope between > the just-produced build and the previous build (from DAG traversal). But > this is obviously scope bloat. I'm not really convinced. Small C++ or rust changes would cause massive diffs. And changes to xul/js would just create the same diff that landed... Only for other peripheral changes this /could/ be useful. Also note that only Linux builds have limited differences when nothing else than MOZ_BUILD_DATE and the changeset differ.
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/2ef8585fcee1 Use the binutils we just built to build GCC. r=gps
Depends on: 1427404
Those failures are due to the now filed bug 1427404.
Flags: needinfo?(mh+mozilla)
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/43d4a4750c0e Use the binutils we just built to build GCC. r=gps
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
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: