Closed
Bug 1199069
Opened 9 years ago
Closed 9 years ago
Remove ancient deopt pragma from AudioStream
Categories
(Core :: Audio/Video: Playback, defect, P2)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla43
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: kinetik, Assigned: kinetik)
Details
Attachments
(1 file)
1.03 KB,
patch
|
jwwang
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=da5ab59f8759
Assignee | ||
Updated•9 years ago
|
Attachment #8653251 -
Flags: review?(jwwang)
Updated•9 years ago
|
Attachment #8653251 -
Flags: review?(jwwang) → review+
Updated•9 years ago
|
Priority: -- → P2
Comment 3•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/22f50375308c
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in
before you can comment on or make changes to this bug.
Description
•