intel / intel/llvm

[SYCL][XPTI] Memory Transfer event instrumentation

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

Description

Hello!

`MemCpyCommand::emitInstrumentationData()`:
https://github.com/intel/llvm/blob/134618fc1f53b5b65876fbb15c3243241c1552e6/sycl/source/detail/scheduler/commands.cpp#L1342-L1347
and
`MemCpyCommandHost::emitInstrumentationData()`:
https://github.com/intel/llvm/blob/134618fc1f53b5b65876fbb15c3243241c1552e6/sycl/source/detail/scheduler/commands.cpp#L1524-L1529
add `copy_from` and `copy_to` metadata as pointer to device.

Is it expected behaviour?

I would expect that device id or device name should be add.

This behaviour was introduced after merge this PR: https://github.com/intel/llvm/pull/4998
@alexbatashev could you take a look please?

**Environment:**
- DPC++ version: 4043dda356af59d3d88607037955a0728dc0f466

**Question**: Is there any way knowing the device ID to get its name?

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.