Closed
Bug 1342514
Opened 7 years ago
Closed 7 years ago
Unable to build FirefoxBeta/Firefox targets with Swift Optimizations on
Categories
(Firefox for iOS :: Build & Test, defect, P3)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
fxios | 8.0+ | --- |
People
(Reporter: sleroux, Assigned: sleroux)
References
Details
(Whiteboard: [MobileCore])
Attachments
(1 file)
After the Swift 3 migration, we're no longer able to compile our distribution targets (Beta/Release) using either Whole Module Optimization or Single-File Optimizations. Setting it single-file shows that the Swift compiler crashes when trying to process Sync's ThreeWayMerger.swift. Marking as blocking/P1 since we can't really submit the app with optimizitions off.
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → sleroux
Status: NEW → ASSIGNED
Assignee | ||
Updated•7 years ago
|
Iteration: --- → 1.16
Assignee | ||
Comment 1•7 years ago
|
||
Building on Xcode 8.3b3 works. Looks like the segfault issue was fixed between 8.2 and 8.3. It might be worth holding off until 8.3 gets released in which case we can just use that for production releases. Closing for now.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Assignee | ||
Comment 2•7 years ago
|
||
Considering that Xcode 8.4 is not out yet and we're shipping next week we should find a way to make this work. I'll take a look today.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Assignee | ||
Comment 3•7 years ago
|
||
Attachment #8846072 -
Flags: review?(rnewman)
Comment 4•7 years ago
|
||
Don't forget to file a bug to re-enable that code :)
Updated•7 years ago
|
Attachment #8846072 -
Flags: review?(rnewman) → review+
Assignee | ||
Comment 5•7 years ago
|
||
Opened follow up bug to uncomment out this code when 8.3 code comes out https://bugzilla.mozilla.org/show_bug.cgi?id=1346370
Updated•7 years ago
|
Updated•7 years ago
|
Iteration: 1.16 → ---
Priority: P1 → P3
Assignee | ||
Comment 6•7 years ago
|
||
This patch only lives on the v7.x branch so we won't need to back this out. Closing this bug since it works on Xcode 8.3.
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•