Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[LIT][ATS] adjust the pi_trace according to ATS trace. #254

Merged
vladimirlaz merged 1 commit into from
May 3, 2021
Merged

[LIT][ATS] adjust the pi_trace according to ATS trace. #254

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion SYCL/Basic/alloc_pinned_host_memory.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ int main() {
}
}

// CHECK:---> piMemBufferCreate
// CHECK:---> piMemBufferCreate
// CHECK-NEXT: {{.*}} : {{.*}}
// CHECK-NEXT: {{.*}} : 9
// CHECK-NEXT: {{.*}} : 17
Copy link

@smaslov-intel smaslov-intel Apr 27, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this number represent and why it changed?

Copy link
Author

@rbegam rbegam Apr 28, 2021 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is the 'Flags' parameter value for piMemBufferCreate. It has chenged because the test is now tracking the trace of the second piMemBufferCreate call (with the use_pinned_host_memory property).
---> piMemBufferCreate(
< unknown > : 0xab0540
< unknown > : 17
< unknown > : 40
< unknown > : 0
< unknown > : 0x7ffda2fa5b08
< nullptr >

Copy link

@smaslov-intel smaslov-intel Apr 28, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see, thanks. LGTM.