Skip to content

Fix #7041: Add charseq registry to CUDA target #7057

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.

Sign up for GitHub

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

Merged
sklam merged 16 commits into from
Nov 19, 2021
Merged

Fix #7041: Add charseq registry to CUDA target #7057

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
1f40bf4
Fix #7041: Add charseq registry to CUDA target
gmarkall May 26, 2021
cf78441
Add type conversion for StringLiteral to unicode_type and test.
stuartarchibald Apr 12, 2021
b52c31e
test_const_string: add expected result for untouched value
gmarkall May 26, 2021
b246c78
Merge remote-tracking branch 'numba/master' into grm-issue-7041
gmarkall Aug 11, 2021
6f8a307
Merge remote-tracking branch 'gmarkall/grm-issue-7041' into grm-issue…
gmarkall Aug 11, 2021
db00c56
Skip const string test on NVVM 3.4
gmarkall Aug 17, 2021
cf6b26f
Don't check whether to skip test_const_string with CUDASIM
gmarkall Aug 20, 2021
3425b23
CUDA: Prevent use of CharSeq with NVVM 3.4
stuartarchibald Nov 15, 2021
c2ad8e2
Focus on CUDA toolkit version in CharSeq message
gmarkall Nov 15, 2021
a44f854
Disallow char sequences with NVVM 3.4
gmarkall Nov 18, 2021
b302af7
Merge remote-tracking branch 'numba/master' into grm-issue-7041
gmarkall Nov 18, 2021
348c019
TestConstString: test with byte strings as well
gmarkall Nov 19, 2021
1634a7f
Add note on CUDA string support to docs
gmarkall Nov 19, 2021
ffe7992
Check for character sequences in record arrays
gmarkall Nov 19, 2021
1784901
Change dtype string from bytes to unicode
gmarkall Nov 19, 2021
b493f48
Remove string type from record in test_record_dtype
gmarkall Nov 19, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Remove string type from record in test_record_dtype 
The string field (`d`) isn't actually read / written in any test here
(probably because that would have generated some code that segfaulted
NVVM 3.4) but is used to exercise alignment of arrays of the record.
Changing the type here to a `uint8` should fulfil this purpose, but
avoid triggering the legalization check with NVVM 3.4
  • Loading branch information
@gmarkall
gmarkall committed Nov 19, 2021
commit b493f48b049da2de87b3a0c8e94cf647540cf87a
2 changes: 1 addition & 1 deletion numba/cuda/tests/cudapy/test_record_dtype.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ def record_read_2d_array(r, a):
('a', np.float64),
('b', np.int32),
('c', np.complex64),
('d', (np.str_, 5))
('d', (np.uint8, 5))
],
align=True
)
Expand Down