Skip to content

Feature/artv3/looptest#324

Merged
trws merged 6 commits intodevelopfrom
feature/artv3/looptest
Sep 12, 2017
Merged

Feature/artv3/looptest#324
trws merged 6 commits intodevelopfrom
feature/artv3/looptest

Conversation

@artv3
Copy link
Copy Markdown
Member

@artv3 artv3 commented Sep 12, 2017

Replaced simd test with loop test.

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #324 into develop will increase coverage by 0.011%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##           develop      #324       +/-   ##
=============================================
+ Coverage   99.016%   99.027%   +0.011%     
=============================================
  Files           44        45        +1     
  Lines         1118      1131       +13     
  Branches        74        77        +3     
=============================================
+ Hits          1107      1120       +13     
  Misses          11        11
Impacted Files Coverage Δ
include/RAJA/policy/sequential/scan.hpp 100% <0%> (ø) ⬆️
include/RAJA/policy/loop/forall.hpp 100% <0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6881e32...5e4c9cb. Read the comment docs.

Copy link
Copy Markdown
Member

@davidbeckingsale davidbeckingsale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea! Thanks @artv3

Copy link
Copy Markdown
Member

@rhornung67 rhornung67 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good @artv3 !

@trws
Copy link
Copy Markdown
Member

trws commented Sep 12, 2017

Much better, thanks @artv3.

@trws trws merged commit 8eea977 into develop Sep 12, 2017
@artv3 artv3 deleted the feature/artv3/looptest branch January 6, 2018 16:58
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.

5 participants