tomflinda comments

Results 62 comments of


                                            tomflinda

Migration of ncclGroupStart and ncclGroupEnd not supported ?

@jinz2014 Current the mappings for ncclGroupStart and ncclGroupEnd are not available in oneCCL side, once the mappings are available, we will plan to support them.

Migrate the raytracing benchmark

@jinz2014 1. On the SYCL side, there is no mapping for cuRAND’s XORWOW. 2. virtual function calls are not supported according to SYCL spec2020, and currently, [the support in intel...

output of __expf()

Hi @zjin-lcf we need to confirm whether there is a precision issue and we will consider to implement it if no precision issue. Thanks.

Use `sycl::bfloat16` class and functions instead of float casts.

@JackAKirk the reference PR #1286 for sentence "The bfloat16 class has been non-experimental for a while now, supporting all backends" is incorrect, could you provide the correct PR? So that...

Use `sycl::bfloat16` class and functions instead of float casts.

@gmlueck @JackAKirk So, after the bfloat16 math functions are merged into base extension, we will plan to refine the migration logic in SYCLomatic, pls remind us after your PR is...

Request: "-I option" to add search include folders

@oscarbg you should specify the option `-in-root` correctly, here is an example: ``` git clone https://github.com/NVIDIA/cuda-samples.git cd cuda-samples\Samples\0_Introduction\vectorAdd dpct -in-root ../../ -out-root out dpct --vcxprojfile=vectorAdd_vs2019.vcxproj --cuda-include-path=/path/to/header file/include/ ``` BTW, -in-root...

error: assigning to 'void ()(const int , int , int)' from incompatible type 'void (const int , int , int, const sycl::nd_item<3> &, int )': different number of parameters (3 vs 5)

@zjin-lcf launching kernel with function pointer is not supported now, while we plan to support it in the future.

no viable conversion from 'typename AccessorCommonT::AccessorSubscript<2 - 1, local_accessor_base<unsigned int, 2, (mode)1026, (placeholder)0>>' to 'volatile uint *'

@jinz2014 reproduced, will do a further investigation.

no viable conversion from 'typename AccessorCommonT::AccessorSubscript<2 - 1, local_accessor_base<unsigned int, 2, (mode)1026, (placeholder)0>>' to 'volatile uint *'

@jinz2014 yes, besides the missing EGS.h, the Makefile also has building issue. I have fixed them manually, now all the migration issues have been identified, we will plan to fix...

unknown type name 'T'

@zjin-lcf The migrated SYCL code with manual fix is as follows : ``` #include #include #include /* DPCT1001:0: The statement could not be removed. */ /* DPCT1000:1: Error handling if-stmt...