Attila Krasznahorkay

Results 49 comments of Attila Krasznahorkay

As we discussed just now in our meeting, here are the ATLAS coding standards. Which could be pointed to, together with those of the other experiments. :wink: http://atlas-computing.web.cern.ch/atlas-computing/projects/qa/draft_guidelines-0.2.html

Aaargh... :angry: No, it's not he buffer after all. It's how the kernel tries to compare the value of `sycl::id` to the `std::size_t` constant. If I remove that, the build...

It does... (Simply removing that one keyword makes the build succeed.) :thinking: I'm always just using `constexpr` together with `static`. Not because of some deep idea behind all of it,...

> Also `CMAKE_SYCL_COMPILER` can be set, avoiding the need for an environment variable. `hipsycl` installs in a way that `CL/sycl.hpp` cannot be found by the vecmem SYCL test program. Not...

Unfortunately I'm not sure what the right approach would be. :frowning: hipSYCL can not be set up just through an environment variable at the moment. This setup that we have...

> > hipSYCL can not be set up just through an environment variable at the moment. > > Not sure I understand what you mean here. I now have vecmem...

:frowning: My strained relationship with hipSYCL continues... How is one supposed to install hipSYCL with spack? :confused: A simple `spack install hipsycl` fails spectacularly on my system. As far as...

Well... ``` [bash][Legolas]:spack > ./spack/bin/spack compiler find ==> Found no new compilers ==> Compilers are defined in the following files: /home/krasznaa/.spack/linux/compilers.yaml [bash][Legolas]:spack > ``` Since spack was not quite turnkey,...

Yepp, this seems to have been fixed in one way or another a long time ago as it seems.

Was done a while ago with: - https://github.com/acts-project/vecmem/blob/main/tests/common/memory_resource_test_alignment.ipp - https://github.com/acts-project/vecmem/blob/main/tests/core/test_core_memory_resources.cpp#L123-L131 Though we may want to still do a little bit more for the "device resources". 🤔 With those only the...