Skip to content

Commit 0e01ed5

Browse files
committed
[SYCL][NFC] Change doxygen comments style
JAVADOC -> C/C++ style Fix one typo: @retun -> \return Signed-off-by: Alexey Bader <[email protected]>
1 parent c72b39c commit 0e01ed5

22 files changed

+595
-594
lines changed

sycl/include/CL/sycl/context.hpp

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ class context {
3232
/// The constructed SYCL context will use the AsyncHandler parameter to handle
3333
/// exceptions.
3434
///
35-
/// @param AsyncHandler is an instance of async_handler.
36-
/// @param UseCUDAPrimaryContext is a bool determining whether to use the
35+
/// \param AsyncHandler is an instance of async_handler.
36+
/// \param UseCUDAPrimaryContext is a bool determining whether to use the
3737
/// primary context in the CUDA backend.
3838
explicit context(const async_handler &AsyncHandler = {},
3939
bool UseCUDAPrimaryContext = false);
@@ -45,9 +45,9 @@ class context {
4545
/// The constructed SYCL context will use the AsyncHandler parameter to handle
4646
/// exceptions.
4747
///
48-
/// @param Device is an instance of SYCL device.
49-
/// @param AsyncHandler is an instance of async_handler.
50-
/// @param UseCUDAPrimaryContext is a bool determining whether to use the
48+
/// \param Device is an instance of SYCL device.
49+
/// \param AsyncHandler is an instance of async_handler.
50+
/// \param UseCUDAPrimaryContext is a bool determining whether to use the
5151
/// primary context in the CUDA backend.
5252
explicit context(const device &Device, async_handler AsyncHandler = {},
5353
bool UseCUDAPrimaryContext = false);
@@ -59,9 +59,9 @@ class context {
5959
/// The constructed SYCL context will use the AsyncHandler parameter to handle
6060
/// exceptions.
6161
///
62-
/// @param Platform is an instance of SYCL platform.
63-
/// @param AsyncHandler is an instance of async_handler.
64-
/// @param UseCUDAPrimaryContext is a bool determining whether to use the
62+
/// \param Platform is an instance of SYCL platform.
63+
/// \param AsyncHandler is an instance of async_handler.
64+
/// \param UseCUDAPrimaryContext is a bool determining whether to use the
6565
/// primary context in the CUDA backend.
6666
explicit context(const platform &Platform, async_handler AsyncHandler = {},
6767
bool UseCUDAPrimaryContext = false);
@@ -74,9 +74,9 @@ class context {
7474
/// The constructed SYCL context will use the AsyncHandler parameter to handle
7575
/// exceptions.
7676
///
77-
/// @param DeviceList is a list of SYCL device instances.
78-
/// @param AsyncHandler is an instance of async_handler.
79-
/// @param UseCUDAPrimaryContext is a bool determining whether to use the
77+
/// \param DeviceList is a list of SYCL device instances.
78+
/// \param AsyncHandler is an instance of async_handler.
79+
/// \param UseCUDAPrimaryContext is a bool determining whether to use the
8080
/// primary context in the CUDA backend.
8181
explicit context(const vector_class<device> &DeviceList,
8282
async_handler AsyncHandler = {},
@@ -88,8 +88,8 @@ class context {
8888
/// The constructed SYCL context will use the AsyncHandler parameter to handle
8989
/// exceptions.
9090
///
91-
/// @param ClContext is an instance of OpenCL cl_context.
92-
/// @param AsyncHandler is an instance of async_handler.
91+
/// \param ClContext is an instance of OpenCL cl_context.
92+
/// \param AsyncHandler is an instance of async_handler.
9393
context(cl_context ClContext, async_handler AsyncHandler = {});
9494

9595
/// Queries this SYCL context for information.
@@ -115,22 +115,22 @@ class context {
115115
///
116116
/// The OpenCL cl_context handle is retained on return.
117117
///
118-
/// @return a valid instance of OpenCL cl_context.
118+
/// \return a valid instance of OpenCL cl_context.
119119
cl_context get() const;
120120

121121
/// Checks if this context is a SYCL host context.
122122
///
123-
/// @return true if this context is a SYCL host context.
123+
/// \return true if this context is a SYCL host context.
124124
bool is_host() const;
125125

126126
/// Gets platform associated with this SYCL context.
127127
///
128-
/// @return a valid instance of SYCL platform.
128+
/// \return a valid instance of SYCL platform.
129129
platform get_platform() const;
130130

131131
/// Gets devices associated with this SYCL context.
132132
///
133-
/// @return a vector of valid SYCL device instances.
133+
/// \return a vector of valid SYCL device instances.
134134
vector_class<device> get_devices() const;
135135

136136
private:

sycl/include/CL/sycl/detail/host_profiling_info.hpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,11 @@ class HostProfilingInfo {
2222
public:
2323
/// Returns event's start time.
2424
///
25-
/// @return event's start time in nanoseconds.
25+
/// \return event's start time in nanoseconds.
2626
cl_ulong getStartTime() const { return StartTime; }
2727
/// Returns event's end time.
2828
///
29-
/// @return event's end time in nanoseconds.
29+
/// \return event's end time in nanoseconds.
3030
cl_ulong getEndTime() const { return EndTime; }
3131

3232
/// Measures event's start time.

sycl/include/CL/sycl/detail/pi.h

Lines changed: 60 additions & 60 deletions
Original file line numberDiff line numberDiff line change
@@ -788,20 +788,20 @@ pi_result piKernelRelease(pi_kernel kernel);
788788
/// Sets up pointer arguments for CL kernels. An extra indirection
789789
/// is required due to CL argument conventions.
790790
///
791-
/// @param kernel is the kernel to be launched
792-
/// @param arg_index is the index of the kernel argument
793-
/// @param arg_size is the size in bytes of the argument (ignored in CL)
794-
/// @param arg_value is the pointer argument
791+
/// \param kernel is the kernel to be launched
792+
/// \param arg_index is the index of the kernel argument
793+
/// \param arg_size is the size in bytes of the argument (ignored in CL)
794+
/// \param arg_value is the pointer argument
795795
pi_result piextKernelSetArgPointer(pi_kernel kernel, pi_uint32 arg_index,
796796
size_t arg_size, const void *arg_value);
797797

798798
/// API to set attributes controlling kernel execution
799799
///
800-
/// @param kernel is the pi kernel to execute
801-
/// @param param_name is a pi_kernel_exec_info value that specifies the info
800+
/// \param kernel is the pi kernel to execute
801+
/// \param param_name is a pi_kernel_exec_info value that specifies the info
802802
/// passed to the kernel
803-
/// @param param_value_size is the size of the value in bytes
804-
/// @param param_value is a pointer to the value to set for the kernel
803+
/// \param param_value_size is the size of the value in bytes
804+
/// \param param_value is a pointer to the value to set for the kernel
805805
///
806806
/// If param_name is PI_USM_INDIRECT_ACCESS, the value will be a ptr to
807807
/// the pi_bool value PI_TRUE
@@ -1025,73 +1025,73 @@ typedef _pi_usm_migration_flags pi_usm_migration_flags;
10251025

10261026
/// Allocates host memory accessible by the device.
10271027
///
1028-
/// @param result_ptr contains the allocated memory
1029-
/// @param context is the pi_context
1030-
/// @param pi_usm_mem_properties are optional allocation properties
1031-
/// @param size_t is the size of the allocation
1032-
/// @param alignment is the desired alignment of the allocation
1028+
/// \param result_ptr contains the allocated memory
1029+
/// \param context is the pi_context
1030+
/// \param pi_usm_mem_properties are optional allocation properties
1031+
/// \param size_t is the size of the allocation
1032+
/// \param alignment is the desired alignment of the allocation
10331033
pi_result piextUSMHostAlloc(void **result_ptr, pi_context context,
10341034
pi_usm_mem_properties *properties, size_t size,
10351035
pi_uint32 alignment);
10361036

10371037
/// Allocates device memory
10381038
///
1039-
/// @param result_ptr contains the allocated memory
1040-
/// @param context is the pi_context
1041-
/// @param device is the device the memory will be allocated on
1042-
/// @param pi_usm_mem_properties are optional allocation properties
1043-
/// @param size_t is the size of the allocation
1044-
/// @param alignment is the desired alignment of the allocation
1039+
/// \param result_ptr contains the allocated memory
1040+
/// \param context is the pi_context
1041+
/// \param device is the device the memory will be allocated on
1042+
/// \param pi_usm_mem_properties are optional allocation properties
1043+
/// \param size_t is the size of the allocation
1044+
/// \param alignment is the desired alignment of the allocation
10451045
pi_result piextUSMDeviceAlloc(void **result_ptr, pi_context context,
10461046
pi_device device,
10471047
pi_usm_mem_properties *properties, size_t size,
10481048
pi_uint32 alignment);
10491049

10501050
/// Allocates memory accessible on both host and device
10511051
///
1052-
/// @param result_ptr contains the allocated memory
1053-
/// @param context is the pi_context
1054-
/// @param device is the device the memory will be allocated on
1055-
/// @param pi_usm_mem_properties are optional allocation properties
1056-
/// @param size_t is the size of the allocation
1057-
/// @param alignment is the desired alignment of the allocation
1052+
/// \param result_ptr contains the allocated memory
1053+
/// \param context is the pi_context
1054+
/// \param device is the device the memory will be allocated on
1055+
/// \param pi_usm_mem_properties are optional allocation properties
1056+
/// \param size_t is the size of the allocation
1057+
/// \param alignment is the desired alignment of the allocation
10581058
pi_result piextUSMSharedAlloc(void **result_ptr, pi_context context,
10591059
pi_device device,
10601060
pi_usm_mem_properties *properties, size_t size,
10611061
pi_uint32 alignment);
10621062

10631063
/// Frees allocated USM memory
10641064
///
1065-
/// @param context is the pi_context of the allocation
1066-
/// @param ptr is the memory to be freed
1065+
/// \param context is the pi_context of the allocation
1066+
/// \param ptr is the memory to be freed
10671067
pi_result piextUSMFree(pi_context context, void *ptr);
10681068

10691069
/// USM Memset API
10701070
///
1071-
/// @param queue is the queue to submit to
1072-
/// @param ptr is the ptr to memset
1073-
/// @param value is value to set. It is interpreted as an 8-bit value and the
1071+
/// \param queue is the queue to submit to
1072+
/// \param ptr is the ptr to memset
1073+
/// \param value is value to set. It is interpreted as an 8-bit value and the
10741074
/// upper
10751075
/// 24 bits are ignored
1076-
/// @param count is the size in bytes to memset
1077-
/// @param num_events_in_waitlist is the number of events to wait on
1078-
/// @param events_waitlist is an array of events to wait on
1079-
/// @param event is the event that represents this operation
1076+
/// \param count is the size in bytes to memset
1077+
/// \param num_events_in_waitlist is the number of events to wait on
1078+
/// \param events_waitlist is an array of events to wait on
1079+
/// \param event is the event that represents this operation
10801080
pi_result piextUSMEnqueueMemset(pi_queue queue, void *ptr, pi_int32 value,
10811081
size_t count, pi_uint32 num_events_in_waitlist,
10821082
const pi_event *events_waitlist,
10831083
pi_event *event);
10841084

10851085
/// USM Memcpy API
10861086
///
1087-
/// @param queue is the queue to submit to
1088-
/// @param blocking is whether this operation should block the host
1089-
/// @param src_ptr is the data to be copied
1090-
/// @param dst_ptr is the location the data will be copied
1091-
/// @param size is number of bytes to copy
1092-
/// @param num_events_in_waitlist is the number of events to wait on
1093-
/// @param events_waitlist is an array of events to wait on
1094-
/// @param event is the event that represents this operation
1087+
/// \param queue is the queue to submit to
1088+
/// \param blocking is whether this operation should block the host
1089+
/// \param src_ptr is the data to be copied
1090+
/// \param dst_ptr is the location the data will be copied
1091+
/// \param size is number of bytes to copy
1092+
/// \param num_events_in_waitlist is the number of events to wait on
1093+
/// \param events_waitlist is an array of events to wait on
1094+
/// \param event is the event that represents this operation
10951095
pi_result piextUSMEnqueueMemcpy(pi_queue queue, pi_bool blocking, void *dst_ptr,
10961096
const void *src_ptr, size_t size,
10971097
pi_uint32 num_events_in_waitlist,
@@ -1100,13 +1100,13 @@ pi_result piextUSMEnqueueMemcpy(pi_queue queue, pi_bool blocking, void *dst_ptr,
11001100

11011101
/// Hint to migrate memory to the device
11021102
///
1103-
/// @param queue is the queue to submit to
1104-
/// @param ptr points to the memory to migrate
1105-
/// @param size is the number of bytes to migrate
1106-
/// @param flags is a bitfield used to specify memory migration options
1107-
/// @param num_events_in_waitlist is the number of events to wait on
1108-
/// @param events_waitlist is an array of events to wait on
1109-
/// @param event is the event that represents this operation
1103+
/// \param queue is the queue to submit to
1104+
/// \param ptr points to the memory to migrate
1105+
/// \param size is the number of bytes to migrate
1106+
/// \param flags is a bitfield used to specify memory migration options
1107+
/// \param num_events_in_waitlist is the number of events to wait on
1108+
/// \param events_waitlist is an array of events to wait on
1109+
/// \param event is the event that represents this operation
11101110
pi_result piextUSMEnqueuePrefetch(pi_queue queue, const void *ptr, size_t size,
11111111
pi_usm_migration_flags flags,
11121112
pi_uint32 num_events_in_waitlist,
@@ -1115,11 +1115,11 @@ pi_result piextUSMEnqueuePrefetch(pi_queue queue, const void *ptr, size_t size,
11151115

11161116
/// USM Memadvise API
11171117
///
1118-
/// @param queue is the queue to submit to
1119-
/// @param ptr is the data to be advised
1120-
/// @param length is the size in bytes of the memory to advise
1121-
/// @param advice is device specific advice
1122-
/// @param event is the event that represents this operation
1118+
/// \param queue is the queue to submit to
1119+
/// \param ptr is the data to be advised
1120+
/// \param length is the size in bytes of the memory to advise
1121+
/// \param advice is device specific advice
1122+
/// \param event is the event that represents this operation
11231123
// USM memadvise API to govern behavior of automatic migration mechanisms
11241124
pi_result piextUSMEnqueueMemAdvise(pi_queue queue, const void *ptr,
11251125
size_t length, int advice, pi_event *event);
@@ -1134,12 +1134,12 @@ pi_result piextUSMEnqueueMemAdvise(pi_queue queue, const void *ptr,
11341134
/// allocation is in bytes. Result is a size_t.
11351135
/// PI_MEM_ALLOC_DEVICE returns the pi_device this was allocated against
11361136
///
1137-
/// @param context is the pi_context
1138-
/// @param ptr is the pointer to query
1139-
/// @param param_name is the type of query to perform
1140-
/// @param param_value_size is the size of the result in bytes
1141-
/// @param param_value is the result
1142-
/// @param param_value_ret is how many bytes were written
1137+
/// \param context is the pi_context
1138+
/// \param ptr is the pointer to query
1139+
/// \param param_name is the type of query to perform
1140+
/// \param param_value_size is the size of the result in bytes
1141+
/// \param param_value is the result
1142+
/// \param param_value_ret is how many bytes were written
11431143
pi_result piextUSMGetMemAllocInfo(pi_context context, const void *ptr,
11441144
pi_mem_info param_name,
11451145
size_t param_value_size, void *param_value,

0 commit comments

Comments
 (0)