Bug 126652 - Make it possible to disable the disabling of AVX
Summary: Make it possible to disable the disabling of AVX
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Filip Pizlo
URL:
Keywords:
Depends on:
Blocks: 112840
  Show dependency treegraph
 
Reported: 2014-01-08 11:56 PST by Filip Pizlo
Modified: 2014-01-30 13:32 PST (History)
7 users (show)

See Also:


Attachments
the patch (2.10 KB, patch)
2014-01-08 11:57 PST, Filip Pizlo
ggaren: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Filip Pizlo 2014-01-08 11:56:11 PST
Yes, disable the disabling.  Why?  Because LLVM's default is to use AVX if it's available, and WebKit can merely just ask LLVM to always avoid AVX even if it's available.  Hence WebKit can disable AVX, and we can disable the disabling of AVX to allow LLVM do just do its default.
Comment 1 Filip Pizlo 2014-01-08 11:57:40 PST
Created attachment 220648 [details]
the patch
Comment 2 Filip Pizlo 2014-01-08 11:59:02 PST
Landed on branch in http://trac.webkit.org/changeset/161515
Comment 3 Geoffrey Garen 2014-01-08 12:43:32 PST
Comment on attachment 220648 [details]
the patch

r=me