Skip to content

Add cl_arm_memory_controls v0.2 extension - #1640

Open
kpet wants to merge 1 commit into
KhronosGroup:mainfrom
kpet:cl-arm-memory-controls
Open

kpet wants to merge 1 commit into
KhronosGroup:mainfrom
kpet:cl-arm-memory-controls

Conversation

@kpet

@kpet kpet commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Kevin Petit <kevin.petit@arm.com>
Change-Id: If1a8bffe6fad12785c726d5cc3ca3bca269fd360

@bashbaug bashbaug left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Feel free to merge this yourself if you'd like, otherwise let me know and I'll click the button.

Comment on lines +74 to +78
typedef enum {
CL_KERNEL_EXEC_INFO_LOCAL_MEMORY_BACKING_GLOBAL_ARM = 1,
CL_KERNEL_EXEC_INFO_LOCAL_MEMORY_BACKING_COMPUTE_UNIT_ARM = 2,
CL_KERNEL_EXEC_INFO_LOCAL_MEMORY_BACKING_AUTO_ARM = 3,
} cl_kernel_exec_info_local_memory_backing_arm;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

FYI, maybe not a big deal, but this does not exactly match what gets generated for the headers.

typedef cl_uint             cl_kernel_exec_info_local_memory_backing_arm;
...
/* cl_kernel_exec_info */
#define CL_KERNEL_EXEC_INFO_LOCAL_MEMORY_BACKING_GLOBAL_ARM 1
#define CL_KERNEL_EXEC_INFO_LOCAL_MEMORY_BACKING_COMPUTE_UNIT_ARM 2
#define CL_KERNEL_EXEC_INFO_LOCAL_MEMORY_BACKING_AUTO_ARM   3

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.

2 participants