-
Notifications
You must be signed in to change notification settings - Fork 12
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
Rework tiles with associator #86
base: main
Are you sure you want to change the base?
Conversation
e57d39e
to
5eba84c
Compare
|
||
// these are now private members | ||
const auto device = alpaka::getDev(queue); | ||
alpaka::memcpy(queue, d_tiles->minMax(), min_max); |
Check warning
Code scanning / Flawfinder (reported by Codacy)
Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. Warning
// these are now private members | ||
const auto device = alpaka::getDev(queue); | ||
alpaka::memcpy(queue, d_tiles->minMax(), min_max); | ||
alpaka::memcpy(queue, d_tiles->tileSize(), tile_sizes); |
Check warning
Code scanning / Flawfinder (reported by Codacy)
Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. Warning
#ifdef DEBUG | ||
alpaka::memcpy(queue_, | ||
alpaka::memcpy(queue, |
Check warning
Code scanning / Flawfinder (reported by Codacy)
Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. Warning
clue::make_device_view(device, dev_points.view()->rho, nPoints)); | ||
alpaka::memcpy(queue_, | ||
clue::make_device_view(device, d_points.view()->rho, nPoints)); | ||
alpaka::memcpy(queue, |
Check warning
Code scanning / Flawfinder (reported by Codacy)
Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. Warning
clue::make_host_view(h_points.debugInfo().nearestHigher.data(), nPoints), | ||
clue::make_device_view(device, dev_points.view()->nearest_higher, nPoints)); | ||
#endif | ||
|
||
alpaka::memcpy(queue_, | ||
alpaka::memcpy(queue, |
Check warning
Code scanning / Flawfinder (reported by Codacy)
Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. Warning
h_view->m_nbins = m_size; | ||
|
||
auto queue(dev); | ||
alpaka::memcpy(queue, m_view, h_view); |
Check warning
Code scanning / Flawfinder (reported by Codacy)
Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. Warning
h_view->m_size = size; | ||
h_view->m_nbins = m_size; | ||
|
||
alpaka::memcpy(queue, m_view, h_view); |
Check warning
Code scanning / Flawfinder (reported by Codacy)
Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. Warning
warp_size); | ||
|
||
auto temp_offsets = make_device_buffer<uint32_t[]>(queue, m_size + 1); | ||
alpaka::memcpy(queue, temp_offsets, m_offsets); |
Check warning
Code scanning / Flawfinder (reported by Codacy)
Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. Warning
host_view->ntiles = n_tiles; | ||
host_view->nperdim = n_perdim; | ||
|
||
alpaka::memcpy(queue, m_view, host_view); |
Check warning
Code scanning / Flawfinder (reported by Codacy)
Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. Warning
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cppcheck (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
clue::make_host_view(h_points.debugInfo().rho.data(), nPoints), | ||
clue::make_device_view(device, dev_points.view()->rho, nPoints)); | ||
alpaka::memcpy(queue_, | ||
alpaka::memcpy(queue, |
Check warning
Code scanning / Flawfinder (reported by Codacy)
Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. Warning
e0e52f2
to
143ac55
Compare
No description provided.