Philip Salzmann

Add new SKIP macro for skipping tests at runtime

12

This adds a new `SKIP` macro for dynamically skipping tests at runtime. The "skipped" status of a test case is treated as a first-class citizen, like "succeeded" or "failed", and...

Multi-GPU + Threads Part 2: The Weirdening

3

Alright so this one is a doozy. I have a minimal-ish test case and a bandaid fix, but I have no idea what is happening. **Bug summary** When interoperating with...

Host buffer not correctly initialized when using const pointer

2

Apparently the host-side buffer is not being correctly initialized with the provided data iff using a `const` pointer: ```cpp #include #include #include int main(int argc, char* argv[]) { const std::vector...

Unit testing

2

While trying to compile more complex applications I often find areas of hipSYCL where I’d like to contribute something, be it a missing operator on some class or error handling...

Decide on which clang-tidy checks we want to enable

1

I've [parsed](https://gist.github.com/psalz/5f936894a451f25e2b9d1d6f66915307) all clang-tidy diagnostics generated with the following checks enabled: ``` -*, bugprone-*, clang-analyzer-*, clang-diagnostic-*, cppcoreguidelines-*, mpi-*, performance-*, readability-* ``` and those are the results: ``` Found 2137 diagnostics...

Replace ::cl::sycl with SYCL 2020 ::sycl namespace

1

SYCL 2020 introduces a new canonical namespace `::sycl`. While the old `::cl::sycl` namespace remains available for backwards compatibility, there is no reason for us to keep using it. The only...

clang-tidy: Enable more checks

1

So far we've used clang-tidy only to enforce naming conventions. This PR enables a whole set of additional checks as suggested in #127. Notably this includes [`misc-const-correctness`](https://clang.llvm.org/extra/clang-tidy/checks/misc/const-correctness.html), a check that...

Mark intrusive_graph_node::has_[dependency|dependent] as const

1

10chars

Add option to launch multiple clang-tidy instances in parallel

3

If a patch touches lots of files the action can actually take quite a while to complete, we are sometimes seeing times upward of 40 minutes. An option to parallelize...