Skip to content

Commit 7bfc276

Browse files
author
Erich Keane
authored
[SYCL] Fix post-commit failure in handler.hpp from unused-parameters. (#3874)
The __builtin_sycl_unique_stable_name patch introduced some changes that @romanovvlad did to make sure we properly called the kernel. However, the changes used the preprocessor to only call the kernel-lambda when necessary. This patch fixes the unused-parameter warnings that happen in host mode by casting these values to 'void' in the else case.
1 parent d3dc212 commit 7bfc276

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

sycl/include/CL/sycl/handler.hpp

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -885,6 +885,8 @@ class __SYCL_EXPORT handler {
885885
#endif
886886
#ifdef __SYCL_DEVICE_ONLY__
887887
KernelFunc();
888+
#else
889+
(void)KernelFunc;
888890
#endif
889891
}
890892

@@ -899,6 +901,9 @@ class __SYCL_EXPORT handler {
899901
#endif
900902
#ifdef __SYCL_DEVICE_ONLY__
901903
KernelFunc(KH);
904+
#else
905+
(void)KernelFunc;
906+
(void)KH;
902907
#endif
903908
}
904909

@@ -913,6 +918,8 @@ class __SYCL_EXPORT handler {
913918
#endif
914919
#ifdef __SYCL_DEVICE_ONLY__
915920
KernelFunc(detail::Builder::getElement(detail::declptr<ElementType>()));
921+
#else
922+
(void)KernelFunc;
916923
#endif
917924
}
918925

@@ -927,6 +934,9 @@ class __SYCL_EXPORT handler {
927934
#endif
928935
#ifdef __SYCL_DEVICE_ONLY__
929936
KernelFunc(detail::Builder::getElement(detail::declptr<ElementType>()), KH);
937+
#else
938+
(void)KernelFunc;
939+
(void)KH;
930940
#endif
931941
}
932942

@@ -941,6 +951,8 @@ class __SYCL_EXPORT handler {
941951
#endif
942952
#ifdef __SYCL_DEVICE_ONLY__
943953
KernelFunc(detail::Builder::getElement(detail::declptr<ElementType>()));
954+
#else
955+
(void)KernelFunc;
944956
#endif
945957
}
946958

@@ -956,6 +968,9 @@ class __SYCL_EXPORT handler {
956968
#endif
957969
#ifdef __SYCL_DEVICE_ONLY__
958970
KernelFunc(detail::Builder::getElement(detail::declptr<ElementType>()), KH);
971+
#else
972+
(void)KernelFunc;
973+
(void)KH;
959974
#endif
960975
}
961976

0 commit comments

Comments
 (0)