Closed Bug 1199069 Opened 5 years ago Closed 5 years ago

Remove ancient deopt pragma from AudioStream

Categories

(Core :: Audio/Video: Playback, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla43
Tracking Status
firefox43 --- fixed

People

(Reporter: kinetik, Assigned: kinetik)

Details

Attachments

(1 file)

Way back in bug 768333, GetPositionInFrames needed a deopt pragma to work around what looked like a miscompilation with MSVC2010 and PGO that resulted in bogus values being returned.  Since the minimum MSVC version (as of Firefox 37) is now MSVC2013, there's no real reason to keep this deopt hack around.
Attachment #8653251 - Flags: review?(jwwang)
Attachment #8653251 - Flags: review?(jwwang) → review+
Priority: -- → P2
https://hg.mozilla.org/mozilla-central/rev/22f50375308c
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.