Skip to content
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

Allow LoopVectorization 0.7 and 0.8. #42

Closed
wants to merge 1 commit into from

Conversation

chriselrod
Copy link

@chriselrod chriselrod commented Jul 29, 2020

This will solve:
https://discourse.julialang.org/t/trying-to-speed-up-laplace-equation-solver/43862/14?u=elrod

If it is possible for the arrays to be empty, then supporting LoopVectorization 0.8 will require adding check_empty=true as an argument to @avx.

Tests passed locally.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 96.36% when pulling 2d378e1 on chriselrod:master into 689d8e2 on PetrKryslUCSD:master.

@PetrKryslUCSD
Copy link
Owner

The package was upgraded to 0.8. Thank you very much.

@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.35%. Comparing base (689d8e2) to head (2d378e1).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #42   +/-   ##
=======================================
  Coverage   96.35%   96.35%           
=======================================
  Files          32       32           
  Lines        3956     3956           
=======================================
  Hits         3812     3812           
  Misses        144      144           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants