Bug 141499

Summary: [Win] [64-bit] Remove MSVC 2013 FMA3 Bug Workaround
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: WebKit Misc.Assignee: peavo
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, peavo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 141498    
Bug Blocks:    
Attachments:
Description Flags
Patch bfulgham: review+

Brent Fulgham
Reported 2015-02-11 21:32:24 PST
At Microsoft's suggestion, we have disabled FMA3 support for 64-bit builds due to a bug in the MSVC 2013 runtime libraries. When we switch to Visual Studio 2015 (or newer), where this bug is fixed, we should remove this workaround so that we can enjoy the improved vector instruction support. See <https://connect.microsoft.com/VisualStudio/feedback/details/811093/visual-studio-2013-rtm-c-x64-code-generation-bug-for-avx2-instructions> for some comments from Microsoft on this topic.
Attachments
Patch (16.99 KB, patch)
2016-03-22 14:07 PDT, peavo
bfulgham: review+
Radar WebKit Bug Importer
Comment 1 2015-02-11 21:33:23 PST
Brent Fulgham
Comment 2 2016-03-22 10:40:44 PDT
Per: If you have a spare minute, would you mind removing this old work-around, now that we are standardized on VS2015? Basically this task is to roll out <http://trac.webkit.org/changeset/179993>.
peavo
Comment 3 2016-03-22 12:05:42 PDT
(In reply to comment #2) > Per: If you have a spare minute, would you mind removing this old > work-around, now that we are standardized on VS2015? > > Basically this task is to roll out <http://trac.webkit.org/changeset/179993>. No problem, I'll create a patch :)
peavo
Comment 4 2016-03-22 14:07:28 PDT
Brent Fulgham
Comment 5 2016-03-22 14:49:31 PDT
Comment on attachment 274683 [details] Patch r=me. Please make sure Windows EWS is happy before landing! :-)
peavo
Comment 6 2016-03-22 14:53:25 PDT
(In reply to comment #5) > Comment on attachment 274683 [details] > Patch > > r=me. Please make sure Windows EWS is happy before landing! :-) Thanks for reviewing!
peavo
Comment 7 2016-03-22 15:21:36 PDT
Note You need to log in before you can comment on or make changes to this bug.