Merge pull request #1017 from IntelPython/change-default-int-type-for-device

oleksandr-pavlyk · web-flow · commit 5510976b6d39 · 2022-12-15T16:48:39.000-06:00
Change default_device_int_type to return 'l' for long.
diff --git a/dpctl/tensor/libtensor/source/device_support_queries.cpp b/dpctl/tensor/libtensor/source/device_support_queries.cpp
@@ -51,7 +51,9 @@ std::string _default_device_fp_type(sycl::device d)
 
 std::string _default_device_int_type(sycl::device)
 {
-    return "i8";
+    return "l"; // code for numpy.dtype('long') to be consisent
+                // with NumPy's default integer type across
+                // platforms.
 }
 
 std::string _default_device_complex_type(sycl::device d)

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,9 @@ std::string _default_device_fp_type(sycl::device d)`
`51`	`51`
`52`	`52`	`std::string _default_device_int_type(sycl::device)`
`53`	`53`	`{`
`54`		`- return "i8";`
	`54`	`+ return "l"; // code for numpy.dtype('long') to be consisent`
	`55`	`+ // with NumPy's default integer type across`
	`56`	`+ // platforms.`
`55`	`57`	`}`
`56`	`58`
`57`	`59`	`std::string _default_device_complex_type(sycl::device d)`