Closed
Bug 1481097
Opened 6 years ago
Closed 6 years ago
Remove workarounds for gcc 4.x bugs
Categories
(Core :: General, enhancement, P3)
Core
General
Tracking
()
People
(Reporter: cpeterson, Assigned: cpeterson)
References
Details
Attachments
(7 files)
(deleted),
text/x-review-board-request
|
glandium
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
djvj
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
dbaron
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
Details | |
(deleted),
text/x-review-board-request
|
lth
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
nika
:
review+
|
Details |
(deleted),
text/x-phabricator-request
|
Details |
These workarounds for gcc 4.x bugs are no longer needed because Firefox currently requires gcc 6.1 or later (as of bug 1444274). Some of these workarounds were for Firefox for Android which now uses clang instead of gcc (as of bug 1163171).
I tested these changes on Try. None of the test failures look related to the removal of these gcc workarounds.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e075db6087e297ee5bc36bbac8c2d3bb8a1d0905
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 9•6 years ago
|
||
mozreview-review |
Comment on attachment 8997753 [details]
Bug 1481097 - wasm: Remove inline template workaround for gcc 4.8.5 bug.
https://reviewboard.mozilla.org/r/261488/#review268494
Attachment #8997753 -
Flags: review?(lhansen) → review+
Comment 10•6 years ago
|
||
mozreview-review |
Comment on attachment 8997751 [details]
Bug 1481097 - layout: Remove nsCSSPseudoElements workaround for gcc 4.9.2 bug.
https://reviewboard.mozilla.org/r/261484/#review268622
Attachment #8997751 -
Flags: review?(dbaron) → review+
Comment 11•6 years ago
|
||
mozreview-review |
Comment on attachment 8997749 [details]
Bug 1481097 - mfbt: Assert we're compiling with gcc 6.1 or later.
https://reviewboard.mozilla.org/r/261480/#review268686
Attachment #8997749 -
Flags: review?(mh+mozilla) → review+
Comment 12•6 years ago
|
||
Pushed by cpeterson@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/20bf59c86755
mfbt: Assert we're compiling with gcc 6.1 or later. r=glandium
https://hg.mozilla.org/integration/mozilla-inbound/rev/8fae317831c1
layout: Remove nsCSSPseudoElements workaround for gcc 4.9.2 bug. r=dbaron
https://hg.mozilla.org/integration/mozilla-inbound/rev/d32128ee9a77
wasm: Remove inline template workaround for gcc 4.8.5 bug. r=lth
Assignee | ||
Updated•6 years ago
|
Keywords: leave-open
Comment 13•6 years ago
|
||
bugherder |
Comment 14•6 years ago
|
||
mozreview-review |
Comment on attachment 8997754 [details]
Bug 1481097 - xpcom: Remove nsTString literal string workaround for gcc 4.9 bug.
https://reviewboard.mozilla.org/r/261490/#review268936
Attachment #8997754 -
Flags: review?(nika) → review+
Comment 15•6 years ago
|
||
Pushed by cpeterson@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/14f33f70fa0c
xpcom: Remove nsTString literal string workaround for gcc 4.9 bug. r=nika
Comment 16•6 years ago
|
||
bugherder |
Comment 17•6 years ago
|
||
Comment on attachment 8997750 [details]
Bug 1481097 - js: Build Array.cpp and JSAtom.cpp in unified mode.
Sorry for the lateness.
Attachment #8997750 -
Flags: review?(kvijayan) → review+
Comment 18•6 years ago
|
||
Pushed by cpeterson@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/eb8f3d884461
js: Build Array.cpp and JSAtom.cpp in unified mode. r=djvj
Assignee | ||
Comment 19•6 years ago
|
||
This gcc 4.8.2 workaround (from bug 1219050) is no longer needed because Firefox currently requires gcc 6.1 or later (as of bug 1444274).
Comment 20•6 years ago
|
||
bugherder |
Comment 21•6 years ago
|
||
Pushed by cpeterson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7e2758590213
vixl: Remove vixl assembler workaround for gcc 4.8.2 bug. r=sstangl
Assignee | ||
Updated•6 years ago
|
status-firefox65:
--- → wontfix
status-firefox66:
--- → wontfix
status-firefox67:
--- → affected
Keywords: leave-open
Comment 22•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Assignee | ||
Comment 23•6 years ago
|
||
Comment on attachment 8997752 [details]
Bug 1481097 - vixl: Remove vixl assembler workaround for gcc 4.8.2 bug.
Removing old review request because Sean r+'d this patch in Phabricator:
https://phabricator.services.mozilla.com/D5506/new/
Attachment #8997752 -
Flags: review?(sstangl)
You need to log in
before you can comment on or make changes to this bug.
Description
•