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

[Kernel] Adds occa::null #270

Merged
merged 1 commit into from
Dec 7, 2019
Merged

[Kernel] Adds occa::null #270

merged 1 commit into from
Dec 7, 2019

Conversation

dmed256
Copy link
Member

@dmed256 dmed256 commented Dec 7, 2019

Description

#235 occa::null can be passed to kernels as a way to omit an occa::memory argument

@dmed256 dmed256 merged commit 7aff480 into master Dec 7, 2019
@dmed256 dmed256 deleted the occa-null branch December 7, 2019 16:42
@codecov
Copy link

codecov bot commented Dec 7, 2019

Codecov Report

Merging #270 into master will increase coverage by 1.36%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #270      +/-   ##
==========================================
+ Coverage    71.7%   73.06%   +1.36%     
==========================================
  Files         282      282              
  Lines       19520    19532      +12     
==========================================
+ Hits        13996    14272     +276     
+ Misses       5524     5260     -264
Impacted Files Coverage Δ
include/occa/core/kernelArg.hpp 100% <100%> (ø) ⬆️
src/core/kernel.cpp 91.41% <100%> (+0.04%) ⬆️
src/core/kernelArg.cpp 81.94% <100%> (+1.34%) ⬆️
src/c/kernel.cpp 1.36% <0%> (-95.9%) ⬇️
src/lang/expr/stringNode.cpp 0% <0%> (-55%) ⬇️
include/occa/tools/hash.hpp 83.33% <0%> (-16.67%) ⬇️
src/c/types.cpp 77.08% <0%> (-6.46%) ⬇️
src/tools/sys.cpp 69.18% <0%> (-5.68%) ⬇️
src/lang/specialMacros.cpp 59.32% <0%> (+0.84%) ⬆️
src/lang/token.cpp 75.11% <0%> (+1.33%) ⬆️
... and 31 more

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.

1 participant