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)

Other
iOS
defect

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: nobody → sleroux
Status: NEW → ASSIGNED
Iteration: --- → 1.16
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
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 → ---
Don't forget to file a bug to re-enable that code :)
Attachment #8846072 - Flags: review?(rnewman) → review+
See Also: → 1346370
Opened follow up bug to uncomment out this code when 8.3 code comes out

https://bugzilla.mozilla.org/show_bug.cgi?id=1346370
Iteration: 1.16 → ---
Priority: P1 → P3
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 ago7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: