[dxil2spv] Remove dxil2spv prototype (#4512)

Revert "[dxil2spv] Print missing runtime errors (#4481)"

This reverts commit 7f8f9ac.

Revert "[dxil2spv] Command line parsing and output file (#4454)"

This reverts commit 2e49e68.

Revert "[dxil2spv] Add additional error checking (#4440)"

This reverts commit 3c1918e.

Revert "[dxil2spv] Add FileCheck testing (#4431)"

This reverts commit d6cec44.

Revert "[dxil2spv] Translate DXIL constants to SPIR-V (#4426)"

This reverts commit f00b3c2.

Revert "[dxil2spv] Translate SV_Position to BuiltIn::Position (#4423)"

This reverts commit b292f94.

Revert "[dxil2spv] Translate extractValue and bufferStore (#4404)"

This reverts commit 1d2a68f.

Revert "[dxil2spv] Translate createHandle and bufferLoad (#4389)"

This reverts commit 316b849.

Revert "[dxil2spv] Support shl instruction (#4388)"

This reverts commit 1039a87.

Revert "[dxil2spv] Fix build warning (#4371)"

This reverts commit 7ecebb3.

Revert "[dxil2spv] Add support for dx.op.threadId (#4354)"

This reverts commit c248a1e.

Revert "[dxil2spv] Add initial compute shader support (#4345)"

This reverts commit c701ece.

Revert "[dxil2spv] Add error checking to file tests (#4344)"

This reverts commit 9f70135.

Revert "[dxil2spv] Add passthrough vertex shader and test (#4337)"

This reverts commit c99f5a5.

Revert "[dxil2spv] SPIR-V validation and related fixes (#4328)"

This reverts commit 4958718.

Revert "[dxil2spv] Implement simple entry function creation (#4323)"

This reverts commit c29a0c7.

Revert "[dxil2spv] Read file directly into llvm::MemoryBuffer (#4304)"

This reverts commit aa26bfd.

Revert "[dxil2spv] Add stage IO variables (#4271)"

This reverts commit 2d5f186.

Revert "[dxil2spv] Add CompilerInstance object (#4229)"

This reverts commit e4cf486.

Revert "[dxil2spv] Add initial testing (#4222)"

This reverts commit c3a61b1.

Revert "[dxil2spv] Construct minimal SPIR-V module (#4216)"

This reverts commit e3da12e.

Revert "[dxil2spv] Add basic dxil2spv CI test (#4209)"

This reverts commit 57d4d2b.

Revert "Add initial dxil2spv executable (#4199)"

This reverts commit c45db48.

Author: sudonatalie

CMakeLists.txt

@@ -136,9 +136,6 @@ endif()
 if (${ENABLE_SPIRV_CODEGEN})
   add_definitions(-DENABLE_SPIRV_CODEGEN)
 endif()
-
-# Building dxil2spv off by default while it's in early development.
-option(ENABLE_DXIL2SPV "Enables DXIL to SPIR-V translation." OFF)
 # SPIRV change ends
 
 include(VersionFromVCS)

appveyor.yml

@@ -40,7 +40,7 @@ before_build:
 build_script:
 - cmd: call utils\hct\hctbuild -%PLATFORM% -%CONFIGURATION% -show-cmake-log -spirvtest
 - sh: mkdir build && cd build
-- sh: cmake .. -GNinja -C ../cmake/caches/PredefinedParams.cmake -DSPIRV_BUILD_TESTS=ON -DCMAKE_BUILD_TYPE=Release -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ -DCMAKE_CXX_FLAGS=-Werror -DENABLE_DXIL2SPV=ON
+- sh: cmake .. -GNinja -C ../cmake/caches/PredefinedParams.cmake -DSPIRV_BUILD_TESTS=ON -DCMAKE_BUILD_TYPE=Release -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ -DCMAKE_CXX_FLAGS=-Werror
 - sh: ninja
 
 test_script:
@@ -52,7 +52,6 @@ test_script:
 - sh: ./bin/dxc -T ps_6_0 -Fo passthru-ps.spv ../tools/clang/test/CodeGenSPIRV/passthru-ps.hlsl2spv -spirv
 - sh: ./bin/clang-spirv-tests --spirv-test-root ../tools/clang/test/CodeGenSPIRV/
 - sh: ./bin/clang-hlsl-tests --HlslDataDir $PWD/../tools/clang/test/HLSL/
-- sh: ./bin/dxil2spv ../tools/clang/test/Dxil2Spv/passthru-ps.ll
 
 after_test:
 # Collect artifacts for Windows
@@ -73,3 +72,4 @@ notifications:
   on_build_success: true
   on_build_failure: true
   on_build_status_changed: true
+

include/dxc/Test/DxcTestUtils.h

@@ -126,10 +126,6 @@ class FileRunCommandPart {
   FileRunCommandResult RunTee(const FileRunCommandResult *Prior);
   FileRunCommandResult RunXFail(const FileRunCommandResult *Prior);
   FileRunCommandResult RunDxilVer(dxc::DxcDllSupport& DllSupport, const FileRunCommandResult* Prior);
-#ifdef ENABLE_DXIL2SPV
-  FileRunCommandResult RunDxil2Spv(dxc::DxcDllSupport &DllSupport,
-                                   const FileRunCommandResult *Prior);
-#endif
   FileRunCommandResult RunDxcHashTest(dxc::DxcDllSupport &DllSupport);
   FileRunCommandResult RunFromPath(const std::string &path, const FileRunCommandResult *Prior);
   FileRunCommandResult RunFileCompareText(const FileRunCommandResult *Prior);

tools/clang/include/clang/SPIRV/SpirvBuilder.h

@@ -52,8 +52,6 @@ class SpirvBuilder {
 public:
   SpirvBuilder(ASTContext &ac, SpirvContext &c, const SpirvCodeGenOptions &,
                FeatureManager &featureMgr);
-  SpirvBuilder(SpirvContext &c, const SpirvCodeGenOptions &,
-               FeatureManager &featureMgr);
   ~SpirvBuilder() = default;
 
   // Forbid copy construction and assignment
@@ -74,9 +72,6 @@ class SpirvBuilder {
   SpirvFunction *createSpirvFunction(QualType returnType, SourceLocation,
                                      llvm::StringRef name, bool isPrecise,
                                      bool isNoInline = false);
-  SpirvFunction *createSpirvFunction(const SpirvType *returnType,
-                                     SourceLocation, llvm::StringRef name,
-                                     bool isPrecise, bool isNoInline = false);
 
   /// \brief Begins building a SPIR-V function by allocating a SpirvFunction
   /// object. Returns the pointer for the function on success. Returns nullptr
@@ -87,10 +82,6 @@ class SpirvBuilder {
                                llvm::StringRef name = "",
                                bool isPrecise = false, bool isNoInline = false,
                                SpirvFunction *func = nullptr);
-  SpirvFunction *beginFunction(const SpirvType *returnType, SourceLocation,
-                               llvm::StringRef name = "",
-                               bool isPrecise = false, bool isNoInline = false,
-                               SpirvFunction *func = nullptr);
 
   /// \brief Creates and registers a function parameter of the given pointer
   /// type in the current function and returns its pointer.
@@ -105,9 +96,6 @@ class SpirvBuilder {
   SpirvVariable *addFnVar(QualType valueType, SourceLocation,
                           llvm::StringRef name = "", bool isPrecise = false,
                           SpirvInstruction *init = nullptr);
-  SpirvVariable *addFnVar(const spirv::SpirvType *valueType, SourceLocation,
-                          llvm::StringRef name = "", bool isPrecise = false,
-                          SpirvInstruction *init = nullptr);
 
   /// \brief Ends building of the current function. All basic blocks constructed
   /// from the beginning or after ending the previous function will be collected
@@ -238,9 +226,6 @@ class SpirvBuilder {
   SpirvBinaryOp *createBinaryOp(spv::Op op, QualType resultType,
                                 SpirvInstruction *lhs, SpirvInstruction *rhs,
                                 SourceLocation loc, SourceRange range = {});
-  SpirvBinaryOp *createBinaryOp(spv::Op op, const SpirvType *resultType,
-                                SpirvInstruction *lhs, SpirvInstruction *rhs,
-                                SourceLocation loc, SourceRange range = {});
 
   SpirvSpecConstantBinaryOp *createSpecConstantBinaryOp(spv::Op op,
                                                         QualType resultType,
@@ -620,10 +605,6 @@ class SpirvBuilder {
   SpirvVariable *addStageIOVar(QualType type, spv::StorageClass storageClass,
                                llvm::StringRef name, bool isPrecise,
                                SourceLocation loc);
-  SpirvVariable *addStageIOVar(const SpirvType *type,
-                               spv::StorageClass storageClass,
-                               llvm::StringRef name, bool isPrecise,
-                               SourceLocation loc);
 
   /// \brief Adds a stage builtin variable whose value is of the given type.
   ///
@@ -633,10 +614,6 @@ class SpirvBuilder {
                                     spv::StorageClass storageClass,
                                     spv::BuiltIn, bool isPrecise,
                                     SourceLocation loc);
-  SpirvVariable *addStageBuiltinVar(const SpirvType *type,
-                                    spv::StorageClass storageClass,
-                                    spv::BuiltIn, bool isPrecise,
-                                    SourceLocation loc);
 
   /// \brief Adds a module variable. This variable should not have the Function
   /// storage class.
@@ -741,12 +718,8 @@ class SpirvBuilder {
   /// and add the context to the list of constants in the module.
   SpirvConstant *getConstantInt(QualType type, llvm::APInt value,
                                 bool specConst = false);
-  SpirvConstant *getConstantInt(const SpirvType *type, llvm::APInt value,
-                                bool specConst = false);
   SpirvConstant *getConstantFloat(QualType type, llvm::APFloat value,
                                   bool specConst = false);
-  SpirvConstant *getConstantFloat(const SpirvType *type, llvm::APFloat value,
-                                  bool specConst = false);
   SpirvConstant *getConstantBool(bool value, bool specConst = false);
   SpirvConstant *
   getConstantComposite(QualType compositeType,
@@ -763,7 +736,6 @@ class SpirvBuilder {
 
 public:
   std::vector<uint32_t> takeModule();
-  std::vector<uint32_t> takeModuleForDxilToSpv();
 
 protected:
   /// Only friend classes are allowed to add capability/extension to the module
@@ -824,7 +796,7 @@ class SpirvBuilder {
                                               SpirvInstruction *var);
 
 private:
-  ASTContext *astContext;
+  ASTContext &astContext;
   SpirvContext &context; ///< From which we allocate various SPIR-V object
   FeatureManager &featureManager;
 

tools/clang/include/clang/SPIRV/SpirvFunction.h

@@ -27,9 +27,6 @@ class SpirvFunction {
   SpirvFunction(QualType astReturnType, SourceLocation,
                 llvm::StringRef name = "", bool precise = false,
                 bool noInline = false);
-  SpirvFunction(const SpirvType *spirvReturnType, SourceLocation,
-                llvm::StringRef name = "", bool precise = false,
-                bool noInline = false);
 
   ~SpirvFunction();
 

tools/clang/include/clang/SPIRV/SpirvInstruction.h

@@ -222,8 +222,6 @@ class SpirvInstruction {
   // Forbid creating SpirvInstruction directly
   SpirvInstruction(Kind kind, spv::Op opcode, QualType astResultType,
                    SourceLocation loc, SourceRange range = {});
-  SpirvInstruction(Kind kind, spv::Op opcode, const SpirvType *resultType,
-                   SourceLocation loc, SourceRange range = {});
 
 protected:
   const Kind kind;
@@ -1021,9 +1019,6 @@ class SpirvBinaryOp : public SpirvInstruction {
   SpirvBinaryOp(spv::Op opcode, QualType resultType, SourceLocation loc,
                 SpirvInstruction *op1, SpirvInstruction *op2,
                 SourceRange range = {});
-  SpirvBinaryOp(spv::Op opcode, const SpirvType *resultType, SourceLocation loc,
-                SpirvInstruction *op1, SpirvInstruction *op2,
-                SourceRange range = {});
 
   DEFINE_RELEASE_MEMORY_FOR_CLASS(SpirvBinaryOp)
 
@@ -1157,8 +1152,6 @@ class SpirvConstantInteger : public SpirvConstant {
 public:
   SpirvConstantInteger(QualType type, llvm::APInt value,
                        bool isSpecConst = false);
-  SpirvConstantInteger(const SpirvType *type, llvm::APInt value,
-                       bool isSpecConst = false);
 
   DEFINE_RELEASE_MEMORY_FOR_CLASS(SpirvConstantInteger)
 
@@ -1181,8 +1174,6 @@ class SpirvConstantFloat : public SpirvConstant {
 public:
   SpirvConstantFloat(QualType type, llvm::APFloat value,
                      bool isSpecConst = false);
-  SpirvConstantFloat(const SpirvType *type, llvm::APFloat value,
-                     bool isSpecConst = false);
 
   DEFINE_RELEASE_MEMORY_FOR_CLASS(SpirvConstantFloat)
 

tools/clang/include/clang/SPIRV/SpirvUtils.h

@@ -16,7 +16,6 @@ add_clang_library(clangSPIRV
   InitListHandler.cpp
   LiteralTypeVisitor.cpp
   LowerTypeVisitor.cpp
-  SpirvTypeVisitor.cpp
   SortDebugInfoVisitor.cpp
   NonUniformVisitor.cpp
   PreciseVisitor.cpp
@@ -33,7 +32,6 @@ add_clang_library(clangSPIRV
   SpirvType.cpp
   SignaturePackingUtil.cpp
   String.cpp
-  SpirvUtils.cpp
 
   LINK_LIBS
   clangAST

tools/clang/lib/SPIRV/CMakeLists.txt

@@ -319,7 +319,7 @@ void EmitVisitor::emitDebugLine(spv::Op op, const SourceLocation &loc,
   }
 
   auto fileId = debugMainFileId;
-  const auto &sm = astContext->getSourceManager();
+  const auto &sm = astContext.getSourceManager();
   const char *fileName = sm.getPresumedLoc(loc).getFilename();
   if (fileName)
     fileId = getOrCreateOpStringId(fileName);

tools/clang/lib/SPIRV/EmitVisitor.cpp

@@ -46,7 +46,7 @@ class EmitTypeHandler {
   };
 
 public:
-  EmitTypeHandler(ASTContext *astCtx, SpirvContext &spvContext,
+  EmitTypeHandler(ASTContext &astCtx, SpirvContext &spvContext,
                   const SpirvCodeGenOptions &opts, FeatureManager &featureMgr,
                   std::vector<uint32_t> *debugVec,
                   std::vector<uint32_t> *decVec,
@@ -145,13 +145,13 @@ class EmitTypeHandler {
   template <unsigned N>
   DiagnosticBuilder emitError(const char (&message)[N],
                               SourceLocation loc = {}) {
-    const auto diagId = astContext->getDiagnostics().getCustomDiagID(
+    const auto diagId = astContext.getDiagnostics().getCustomDiagID(
         clang::DiagnosticsEngine::Error, message);
-    return astContext->getDiagnostics().Report(loc, diagId);
+    return astContext.getDiagnostics().Report(loc, diagId);
   }
 
 private:
-  ASTContext *astContext;
+  ASTContext &astContext;
   SpirvContext &context;
   FeatureManager featureManager;
   std::vector<uint32_t> curTypeInst;
@@ -198,10 +198,9 @@ class EmitVisitor : public Visitor {
   };
 
 public:
-  EmitVisitor(ASTContext *astCtx, SpirvContext &spvCtx,
+  EmitVisitor(ASTContext &astCtx, SpirvContext &spvCtx,
               const SpirvCodeGenOptions &opts, FeatureManager &featureMgr)
-      : Visitor(opts, spvCtx), astContext(astCtx), featureManager(featureMgr),
-        id(0),
+      : Visitor(opts, spvCtx), astContext(astCtx), featureManager(featureMgr), id(0),
         typeHandler(astCtx, spvCtx, opts, featureMgr, &debugVariableBinary,
                     &annotationsBinary, &typeConstantBinary,
                     [this]() -> uint32_t { return takeNextId(); }),
@@ -396,14 +395,14 @@ class EmitVisitor : public Visitor {
   template <unsigned N>
   DiagnosticBuilder emitError(const char (&message)[N],
                               SourceLocation loc = {}) {
-    const auto diagId = astContext->getDiagnostics().getCustomDiagID(
+    const auto diagId = astContext.getDiagnostics().getCustomDiagID(
         clang::DiagnosticsEngine::Error, message);
-    return astContext->getDiagnostics().Report(loc, diagId);
+    return astContext.getDiagnostics().Report(loc, diagId);
   }
 
 private:
   // Object that holds Clang AST nodes.
-  ASTContext *astContext;
+  ASTContext &astContext;
   // Feature manager.
   FeatureManager featureManager;
   // The last result-id that's been used so far.