Make DXC and DXV use new DxcDllExtValidationLoader to load external validator before validation.

azure-pipelines.yml

@@ -45,7 +45,23 @@ stages:
     - script: |
         call utils\hct\hctstart.cmd %HLSL_SRC_DIR% %HLSL_BLD_DIR%
         call utils\hct\hcttest.cmd -$(configuration) exec
-      displayName: 'DXIL Execution Tests'
+      displayName: 'DXIL Execution Tests'        
+    - script: |
+        call utils\hct\hctstart.cmd %HLSL_SRC_DIR% %HLSL_BLD_DIR%
+        call utils\hct\hcttest.cmd -$(configuration) compat-suite 1.6
+      displayName: 'DXIL Compat Suite Tests (1.6 release)'
+      condition: succeededOrFailed()
+    - script: |
+        call utils\hct\hctstart.cmd %HLSL_SRC_DIR% %HLSL_BLD_DIR%
+        call utils\hct\hcttest.cmd -$(configuration) compat-suite 1.7
+      displayName: 'DXIL Compat Suite Tests (1.7 release)'
+      condition: succeededOrFailed()
+    - script: |
+        call utils\hct\hctstart.cmd %HLSL_SRC_DIR% %HLSL_BLD_DIR%
+        call utils\hct\hcttest.cmd -$(configuration) compat-suite 1.8
+      displayName: 'DXIL Compat Suite Tests (1.8 release)'
+      condition: succeededOrFailed()
+
 
   - job: Nix
     timeoutInMinutes: 165

include/dxc/Support/HLSLOptions.h

@@ -227,6 +227,7 @@ class DxcOpts {
   std::string TimeTrace = "";           // OPT_ftime_trace[EQ]
   unsigned TimeTraceGranularity = 500;  // OPT_ftime_trace_granularity_EQ
   bool VerifyDiagnostics = false;       // OPT_verify
+  bool Verbose = false;                 // OPT_verbose
 
   // Optimization pass enables, disables and selects
   OptimizationToggles

include/dxc/Support/HLSLOptions.td

@@ -197,6 +197,18 @@ def verify : Joined<["-"], "verify">,
   Group<hlslcomp_Group>, Flags<[CoreOption, DriverOption]>,
   HelpText<"Verify diagnostic output using comment directives">;
 
+def verbose : Flag<["-"], "verbose">,
+              Group<hlslcomp_Group>,
+              Flags<[DriverOption]>,
+              HelpText<"Allow emission of verbose compiler information">;
+
+// Short alias '-v' for the verbose option
+def v : Flag<["-"], "v">,
+        Alias<verbose>,
+        Group<hlslcomp_Group>,
+        Flags<[DriverOption]>,
+        HelpText<"Alias for -verbose">;
+
 /*
 def fno_caret_diagnostics : Flag<["-"], "fno-caret-diagnostics">, Group<hlslcomp_Group>,
  Flags<[CoreOption]>;

include/dxc/Support/dxcapi.extval.h

@@ -1,8 +1,12 @@
 #include "dxc/Support/dxcapi.use.h"
+#include "dxc/WinAdapter.h"
+#include "llvm/Support/raw_ostream.h"
+
 #include <cassert>
 #include <string>
 
 namespace dxc {
+
 class DxcDllExtValidationLoader : public DllLoader {
   // DxCompilerSupport manages the
   // lifetime of dxcompiler.dll, while DxilExtValSupport
@@ -12,8 +16,8 @@ class DxcDllExtValidationLoader : public DllLoader {
   std::string DxilDllPath;
 
 public:
-  std::string GetDxilDllPath() { return DxilDllPath; }
-  bool DxilDllFailedToLoad() {
+  std::string getDxilDllPath() { return DxilDllPath; }
+  bool dxilDllFailedToLoad() {
     return !DxilDllPath.empty() && !DxilExtValSupport.IsEnabled();
   }
 
@@ -22,7 +26,7 @@ class DxcDllExtValidationLoader : public DllLoader {
   HRESULT CreateInstance2Impl(IMalloc *pMalloc, REFCLSID clsid, REFIID riid,
                               IUnknown **pResult) override;
 
-  HRESULT Initialize();
+  HRESULT initialize(llvm::raw_string_ostream &log);
 
   bool IsEnabled() const override { return DxCompilerSupport.IsEnabled(); }
 };

lib/DxcSupport/HLSLOptions.cpp

@@ -871,6 +871,7 @@ int ReadDxcOpts(const OptTable *optionTable, unsigned flagsToInclude,
   opts.TimeReport = Args.hasFlag(OPT_ftime_report, OPT_INVALID, false);
   opts.TimeTrace = Args.hasFlag(OPT_ftime_trace, OPT_INVALID, false) ? "-" : "";
   opts.VerifyDiagnostics = Args.hasFlag(OPT_verify, OPT_INVALID, false);
+  opts.Verbose = Args.hasFlag(OPT_verbose, OPT_INVALID, false);
   if (Args.hasArg(OPT_ftime_trace_EQ))
     opts.TimeTrace = Args.getLastArgValue(OPT_ftime_trace_EQ);
   if (Arg *A = Args.getLastArg(OPT_ftime_trace_granularity_EQ)) {