Skip to content

Commit

Permalink
Fix regex working-memory-size refactor error (#12119)
Browse files Browse the repository at this point in the history
Fixes error in `working_memory_size()` member function passing the parameters incorrectly.
This was introduce in #11927 and found in the nightly compute-sanitizer check.
https://gpuci.gpuopenanalytics.com/job/rapidsai/job/gpuci/job/cudf/job/branches/job/cudf-gpu-build-branch-22.12/19/CUDA=11.5/testReport/junit/cudamemcheck/STRINGS_TEST/StringsContainsTests_ContainsTest/

Authors:
  - David Wendt (https://github.com/davidwendt)

Approvers:
  - Jason Lowe (https://github.com/jlowe)
  - Yunsong Wang (https://github.com/PointKernel)
  - Nghia Truong (https://github.com/ttnghia)
  - Vukasin Milovanovic (https://github.com/vuule)

URL: #12119
  • Loading branch information
davidwendt authored Nov 10, 2022
1 parent b30664b commit 7f2a471
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cpp/src/strings/regex/regexec.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ void reprog_device::destroy() { delete this; }

std::size_t reprog_device::working_memory_size(int32_t num_threads) const
{
return compute_working_memory_size(insts_counts(), num_threads);
return compute_working_memory_size(num_threads, insts_counts());
}

std::pair<std::size_t, int32_t> reprog_device::compute_strided_working_memory(
Expand Down

0 comments on commit 7f2a471

Please sign in to comment.