Skip to content

Commit a8afe1d

Browse files
committed
Address more PR feedback
Signed-off-by: Lukas Sommer <[email protected]>
1 parent 75a77fd commit a8afe1d

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

sycl-fusion/jit-compiler/lib/translation/KernelTranslation.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -220,6 +220,9 @@ llvm::Expected<KernelBinary *>
220220
KernelTranslator::translateToPTX(SYCLKernelInfo &KernelInfo, llvm::Module &Mod,
221221
JITContext &JITCtx) {
222222
#ifndef FUSION_JIT_SUPPORT_PTX
223+
(void)KernelInfo;
224+
(void)Mod;
225+
(void)JITCtx;
223226
return createStringError(inconvertibleErrorCode(),
224227
"PTX translation not supported in this build");
225228
#else // FUSION_JIT_SUPPORT_PTX

sycl-fusion/passes/target/TargetFusionInfo.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,8 @@ class TargetFusionInfoImpl {
4949
llvm::Module *LLVMMod;
5050
};
5151

52+
namespace {
53+
5254
//
5355
// SPIRVTargetFusionInfo
5456
//
@@ -282,6 +284,8 @@ class NVPTXTargetFusionInfo : public TargetFusionInfoImpl {
282284
};
283285
#endif // FUSION_JIT_SUPPORT_PTX
284286

287+
} // anonymous namespace
288+
285289
//
286290
// TargetFusionInfo
287291
//

0 commit comments

Comments
 (0)