Skip to content

Commit c320bc5

Browse files
[NFCI][SYCL] Minor code cleanup
1 parent 7d40de9 commit c320bc5

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

sycl/include/sycl/detail/cg_types.hpp

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -164,11 +164,7 @@ class HostKernelBase {
164164
// Class which stores specific lambda object.
165165
template <class KernelType, class KernelArgType, int Dims>
166166
class HostKernel : public HostKernelBase {
167-
using IDBuilder = sycl::detail::Builder;
168167
KernelType MKernel;
169-
// Allowing accessing MKernel from 'ResetHostKernelHelper' method of
170-
// 'sycl::handler'
171-
friend class sycl::handler;
172168

173169
public:
174170
HostKernel(KernelType Kernel) : MKernel(Kernel) {}
@@ -182,6 +178,7 @@ class HostKernel : public HostKernelBase {
182178
// kernel code instructions with source code lines.
183179
// NOTE: InstatiateKernelOnHost() should not be called.
184180
void InstantiateKernelOnHost() override {
181+
using IDBuilder = sycl::detail::Builder;
185182
if constexpr (std::is_same_v<KernelArgType, void>) {
186183
runKernelWithoutArg(MKernel);
187184
} else if constexpr (std::is_same_v<KernelArgType, sycl::id<Dims>>) {

sycl/include/sycl/handler.hpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -750,7 +750,7 @@ class __SYCL_EXPORT handler {
750750
#endif
751751
// Empty name indicates that the compilation happens without integration
752752
// header, so don't perform things that require it.
753-
if (KernelHasName) {
753+
if constexpr (KernelHasName) {
754754
// TODO support ESIMD in no-integration-header case too.
755755
clearArgs();
756756
extractArgsAndReqsFromLambda(MHostKernel->getPtr(),
@@ -767,7 +767,7 @@ class __SYCL_EXPORT handler {
767767

768768
// If the kernel lambda is callable with a kernel_handler argument, manifest
769769
// the associated kernel handler.
770-
if (IsCallableWithKernelHandler) {
770+
if constexpr (IsCallableWithKernelHandler) {
771771
getOrInsertHandlerKernelBundle(/*Insert=*/true);
772772
}
773773
}

0 commit comments

Comments
 (0)