[PATCH] D141705: [HLSL] [Dirver] add dxv as a Driver Action Job
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Jan 29 12:37:24 PST 2023
MaskRay added inline comments.
================
Comment at: clang/include/clang/Basic/DiagnosticDriverKinds.td:687
+def warn_drv_dxc_missing_dxv : Warning<"dxv not found."
+ " Resulting DXIL will not be signed for use in release environments.">;
----------------
python3kgae wrote:
> beanz wrote:
> > Not all `dxv` binaries can sign… some just validate. Only the “official” releases support signing.
> Fixed.
Make sure there is a space after `.`. The diagnostics convention does not include the trailing period.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141705/new/
https://reviews.llvm.org/D141705
More information about the cfe-commits
mailing list