Skip to content

Conversation

@againull
Copy link
Contributor

  • Introduce pi_buffer_region type and use it instead of cl_buffer_region
  • Introduce pi_fp_capabilities enum and use it in PI functions
  • Fix type of mem advice parameter in piextUSMEnqueueMemAdvise
  • Use pi_event_info instead of cl_event_info
  • Extend pi_device_info with subgroup properties
  • Extend pi_device_info with enum values to query subgroup information
    and IL version, this values are going to be used level zero plugin

Signed-off-by: Artur Gainullin [email protected]

@againull againull requested review from a team and smaslov-intel as code owners May 19, 2020 19:40
@againull againull requested a review from sergey-semenov May 19, 2020 19:40
* Introduce pi_buffer_region type and use it instead of cl_buffer_region
* Introduce pi_fp_capabilities enum and use it in PI functions
* Fix type of mem advice parameter in piextUSMEnqueueMemAdvise
* Use pi_event_info instead of cl_event_info
* Extend pi_device_info with subgroup properties
* Extend pi_device_info with enum values to query subgroup information
and IL version, this values are going to be used level zero plugin

Signed-off-by: Artur Gainullin <[email protected]>
@againull againull force-pushed the untie_from_opencl branch from bc4f5c3 to 249e57b Compare May 19, 2020 20:27
@smaslov-intel
Copy link
Contributor

@againull , please do not force-push as it kills the ability to do incremental review

smaslov-intel
smaslov-intel previously approved these changes May 19, 2020
Signed-off-by: Artur Gainullin <[email protected]>
size_t size;
};
using pi_buffer_region_struct *pi_buffer_region;
using pi_buffer_region = pi_buffer_region_struct *;
Copy link
Contributor

Choose a reason for hiding this comment

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

BTW, all of pi.h is meant to be C-only interface

@bader bader merged commit 358ae27 into intel:sycl May 20, 2020
@againull againull deleted the untie_from_opencl branch December 3, 2022 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants