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

Conversation

@rbegam
Copy link

@rbegam rbegam commented Apr 27, 2021

Signed-off-by: rbegam rehana.begam@intel.com

Signed-off-by: rbegam <rehana.begam@intel.com>
@rbegam rbegam requested a review from a team as a code owner April 27, 2021 18:28
// CHECK:---> piMemBufferCreate
// CHECK-NEXT: {{.*}} : {{.*}}
// CHECK-NEXT: {{.*}} : 9
// CHECK-NEXT: {{.*}} : 17

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

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 >

Choose a reason for hiding this comment

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

I see, thanks. LGTM.

@rbegam
Copy link
Author

rbegam commented Apr 30, 2021

@vladimirlaz need to merge this. thanks.

@vladimirlaz vladimirlaz merged commit e0922f0 into intel:intel May 3, 2021
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants