intel / intel/llvm

Refactor SYCLBIN testing to unify target modes

Open
#20,736 0 comments 0 reactions 0 assignees View on GitHub
confirmed enhancement
Dominant language
LLVM
Stars
1.5k
Forks
854
Avg merge
3d 17h
Merged PRs (30d)
137

Description

Suggested by @aelovikov-intel in https://github.com/intel/llvm/pull/20707#pullrequestreview-3493514286_:

Have a local format.py in SYCLBIN such that

```c++
// REQUIRES-EXEC: ...
// REQUIRES-OBJ: ...
// REQUIRES-INPUT: ...
// RUN-EXEC:
// RUN-OBJ:
// RUN-INPUT:
// RUN: %{build} ...
// RUN: %{run} ...
```

would generate three tests "on-the-fly".

This should allow the SYCLBIN tests to include `execution`, `object` and `input` testing in the same file, rather than having to split each case into separate files, despite testing the same functionality for the different target modes.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.