Skip to content

Commit 277f315

Browse files
authored
[SYCL] Fix warnings in libdevice (#1630)
Apply the `DEVICE_EXERNAL` attribute also to the function definitions, not only to the function declarations. Signed-off-by: Andrea Bocci <[email protected]>
1 parent bd5f71b commit 277f315

File tree

2 files changed

+16
-16
lines changed

2 files changed

+16
-16
lines changed

libdevice/device.h

+4-4
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,6 @@
99
#ifndef __LIBDEVICE_DEVICE_H__
1010
#define __LIBDEVICE_DEVICE_H__
1111

12-
// We need the following header to ensure the definition of all spirv variables
13-
// required by the wrapper libraries.
14-
#include "spirv_vars.hpp"
15-
1612
#ifdef __cplusplus
1713
#define EXTERN_C extern "C"
1814
#else // __cplusplus
@@ -37,4 +33,8 @@
3733

3834
#define DEVICE_EXTERN_C DEVICE_EXTERNAL EXTERN_C
3935

36+
// We need the following header to ensure the definition of all spirv variables
37+
// required by the wrapper libraries.
38+
#include "spirv_vars.hpp"
39+
4040
#endif // __LIBDEVICE_DEVICE_H__

libdevice/spirv_vars.hpp

+12-12
Original file line numberDiff line numberDiff line change
@@ -15,37 +15,37 @@
1515

1616
#ifdef __SYCL_NVPTX__
1717

18-
SYCL_EXTERNAL size_t __spirv_GlobalInvocationId_x();
19-
SYCL_EXTERNAL size_t __spirv_GlobalInvocationId_y();
20-
SYCL_EXTERNAL size_t __spirv_GlobalInvocationId_z();
18+
DEVICE_EXTERNAL size_t __spirv_GlobalInvocationId_x();
19+
DEVICE_EXTERNAL size_t __spirv_GlobalInvocationId_y();
20+
DEVICE_EXTERNAL size_t __spirv_GlobalInvocationId_z();
2121

22-
SYCL_EXTERNAL size_t __spirv_LocalInvocationId_x();
23-
SYCL_EXTERNAL size_t __spirv_LocalInvocationId_y();
24-
SYCL_EXTERNAL size_t __spirv_LocalInvocationId_z();
22+
DEVICE_EXTERNAL size_t __spirv_LocalInvocationId_x();
23+
DEVICE_EXTERNAL size_t __spirv_LocalInvocationId_y();
24+
DEVICE_EXTERNAL size_t __spirv_LocalInvocationId_z();
2525

2626
#else // __SYCL_NVPTX__
2727

2828
typedef size_t size_t_vec __attribute__((ext_vector_type(3)));
2929
extern "C" const size_t_vec __spirv_BuiltInGlobalInvocationId;
3030
extern "C" const size_t_vec __spirv_BuiltInLocalInvocationId;
3131

32-
SYCL_EXTERNAL inline size_t __spirv_GlobalInvocationId_x() {
32+
DEVICE_EXTERNAL inline size_t __spirv_GlobalInvocationId_x() {
3333
return __spirv_BuiltInGlobalInvocationId.x;
3434
}
35-
SYCL_EXTERNAL inline size_t __spirv_GlobalInvocationId_y() {
35+
DEVICE_EXTERNAL inline size_t __spirv_GlobalInvocationId_y() {
3636
return __spirv_BuiltInGlobalInvocationId.y;
3737
}
38-
SYCL_EXTERNAL inline size_t __spirv_GlobalInvocationId_z() {
38+
DEVICE_EXTERNAL inline size_t __spirv_GlobalInvocationId_z() {
3939
return __spirv_BuiltInGlobalInvocationId.z;
4040
}
4141

42-
SYCL_EXTERNAL inline size_t __spirv_LocalInvocationId_x() {
42+
DEVICE_EXTERNAL inline size_t __spirv_LocalInvocationId_x() {
4343
return __spirv_BuiltInLocalInvocationId.x;
4444
}
45-
SYCL_EXTERNAL inline size_t __spirv_LocalInvocationId_y() {
45+
DEVICE_EXTERNAL inline size_t __spirv_LocalInvocationId_y() {
4646
return __spirv_BuiltInLocalInvocationId.y;
4747
}
48-
SYCL_EXTERNAL inline size_t __spirv_LocalInvocationId_z() {
48+
DEVICE_EXTERNAL inline size_t __spirv_LocalInvocationId_z() {
4949
return __spirv_BuiltInLocalInvocationId.z;
5050
}
5151

0 commit comments

Comments
 (0)