diff --git a/clang/lib/CodeGen/CGHLSLRuntime.cpp b/clang/lib/CodeGen/CGHLSLRuntime.cpp index 8e0088784e5485..c61a65813aea51 100644 --- a/clang/lib/CodeGen/CGHLSLRuntime.cpp +++ b/clang/lib/CodeGen/CGHLSLRuntime.cpp @@ -143,6 +143,7 @@ void CGHLSLRuntime::emitEntryFunction(const FunctionDecl *FD, IRBuilder<> B(BB); llvm::SmallVector Args; // FIXME: support struct parameters where semantics are on members. + // See: https://github.com/llvm/llvm-project/issues/57874 for (const auto *Param : FD->parameters()) { Args.push_back(emitInputSemantic(B, *Param)); } @@ -150,6 +151,7 @@ void CGHLSLRuntime::emitEntryFunction(const FunctionDecl *FD, CallInst *CI = B.CreateCall(FunctionCallee(Fn), Args); (void)CI; // FIXME: Handle codegen for return type semantics. + // See: https://github.com/llvm/llvm-project/issues/57875 B.CreateRetVoid(); } diff --git a/clang/lib/Driver/ToolChains/HLSL.cpp b/clang/lib/Driver/ToolChains/HLSL.cpp index a80fea2e3efe25..69a78209462466 100644 --- a/clang/lib/Driver/ToolChains/HLSL.cpp +++ b/clang/lib/Driver/ToolChains/HLSL.cpp @@ -209,5 +209,6 @@ HLSLToolChain::TranslateArgs(const DerivedArgList &Args, StringRef BoundArch, } // FIXME: add validation for enable_16bit_types should be after HLSL 2018 and // shader model 6.2. + // See: https://github.com/llvm/llvm-project/issues/57876 return DAL; } diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index 0f567d91dc81ef..865388ccd5fa80 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -11896,6 +11896,7 @@ void Sema::CheckHLSLEntryPoint(FunctionDecl *FD) { for (const auto *Param : FD->parameters()) { if (!Param->hasAttr()) { // FIXME: Handle struct parameters where annotations are on struct fields. + // See: https://github.com/llvm/llvm-project/issues/57875 Diag(FD->getLocation(), diag::err_hlsl_missing_semantic_annotation); Diag(Param->getLocation(), diag::note_previous_decl) << Param; FD->setInvalidDecl(); diff --git a/clang/test/CodeGenHLSL/validator_version.hlsl b/clang/test/CodeGenHLSL/validator_version.hlsl index 0d2cbb96dbe483..a3fc4fa1b63c4f 100644 --- a/clang/test/CodeGenHLSL/validator_version.hlsl +++ b/clang/test/CodeGenHLSL/validator_version.hlsl @@ -1,6 +1,7 @@ // RUN: %clang -cc1 -S -triple dxil-pc-shadermodel6.3-library -S -emit-llvm -xhlsl -validator-version 1.1 -o - %s | FileCheck %s // FIXME:The following line should work once SPIR-V support for HLSL is added. +// See: https://github.com/llvm/llvm-project/issues/57877 // DISABLED: %clang -cc1 -S -triple spirv32 -S -emit-llvm -xhlsl -validator-version 1.1 -o - %s | FileCheck %s --check-prefix=NOT_DXIL // CHECK:!dx.valver = !{![[valver:[0-9]+]]}