Hi Tanya,
The patch looks good, but I wondered if we should separate all the
-cl-kernel-arg-info related metadata somehow from the rest of the kernel
metadata, something like:
!1 = metadata !{metadata !"kernel_arg_info", metadata !2, metadata !3,...
!2 = metadata !{metadata !"kernel_arg_name", metadata !"X", metadata !"Y",
metadata !"anotherArg"}
!3 = metadata !{metadata !"kernel_arg_type_qualifier", i32 0, i32 0, i32 3}
Thanks
Guy
-----Original Message-----
From: [email protected] [mailto:[email protected]]
On Behalf Of Tanya Lattner
Sent: Tuesday, June 26, 2012 02:00
To: [email protected]
Subject: [cfe-commits] [PATCH] OpenCL Kernel arg info metadata
Attached is a patch that implements the OpenCL kernel arg info option
(cl-kernel-arg-info) and during code generation it outputs metadata for kernel
arguments. In order to have incremental changes, this only adds the kernel arg
names to the metadata. The format of the metadata matches the criteria that was
discussed early on the mailing list in the form of lists where the first is the
"name" of the type of metadata and N number of Values. This allows for easy
extension in the future and other OpenCL implementations to do their own thing.
More info in the comments.
I've included a test case.
Please review.
-Tanya
---------------------------------------------------------------------
Intel Israel (74) Limited
This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
