-
Notifications
You must be signed in to change notification settings - Fork 4.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use CMS_USE_AVX2 instead of CMS_USE_AVX #33268
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33268/21759
|
A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for master. It involves the following packages: DataFormats/Math @perrotta, @jpata, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-ccae58/13761/summary.html Comparison SummarySummary:
|
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Fixes #33267
SSEVec.h
has code forAVX2
and does not work when build foravx
e.gsandybridge
( see #33267). This PR proposes to defineCMS_USE_AVX2
instead ofCMS_USE_AVX
when__AVX2__
is available.This change should not have any effects of normal cmssw builds as we build with
-msse3
however this should fix theSKYLAKEAVX512
IBs which are now build forsandybridge,haswell,skylake-avx512