cms-patatrack / pixeltrack-standalone

Standalone Patatrack pixel tracking
Apache License 2.0
17 stars 35 forks source link

[sycltest] Update Makefile and wait synchronously explicitly #345

Closed makortel closed 2 years ago

makortel commented 2 years ago

I gave a try on sycltest with dpcpp compiled manually with the CUDA support enabled. I modified a bit the Makefile variables to make the dpct dependence optional (although the -isystem $(DPCT_BASE)/include didn't do any practical harm in my case), and added USER_SYCLFLAGS to add custom compilation (even if currently USER_CXXFLAGS would have done the same).

The compiler complained about not capturing the return value of std::async(), so for time being I removed the call and made the ScopedContextHolderHelper::enqueueCallback() explicitly synchronous. This was the behavior effectively earlier too, since the destructor of std::future returned by std::async() blocks until the asynchronous work is complete.