Moved shader compile logic to main libary

This commit is contained in:
Alejandro Saucedo 2021-02-20 16:48:56 +00:00
parent f713b56862
commit 9a19e3b280
13 changed files with 197 additions and 91 deletions

View file

@ -16,6 +16,7 @@ option(KOMPUTE_OPT_INSTALL "Enable if you want to enable installation" 0)
# Build options
option(KOMPUTE_OPT_BUILD_PYTHON "Enable if you want to build python bindings" 0)
option(KOMPUTE_OPT_ENABLE_SPDLOG "Extra compile flags for Kompute, see docs for full list" 0)
option(KOMPUTE_OPT_DISABLE_GLSLANG "Skip compilation of glslang utilities" 0)
option(KOMPUTE_OPT_REPO_SUBMODULE_BUILD, "Use the submodule repos instead of external package manager" 0)
option(KOMPUTE_OPT_ANDOID_BUILD "Enable android compilation flags required" 0)
option(KOMPUTE_OPT_DISABLE_VK_DEBUG_LAYERS "Explicitly disable debug layers even on debug" 0)
@ -24,7 +25,10 @@ set(KOMPUTE_EXTRA_CXX_FLAGS "" CACHE STRING "Extra compile flags for Kompute, se
if(KOMPUTE_OPT_ENABLE_SPDLOG)
set(KOMPUTE_EXTRA_CXX_FLAGS "${KOMPUTE_EXTRA_CXX_FLAGS} -DKOMPUTE_ENABLE_SPDLOG=1")
set(SPDLOG_INSTALL, 1)
if(KOMPUTE_OPT_INSTALL)
# Enable install parameters for spdlog (overrides parameters passed)
set(SPDLOG_INSTALL ON CACHE BOOL "Enables install of glslang" FORCE)
endif()
endif()
if(KOMPUTE_OPT_ANDOID_BUILD)
@ -39,6 +43,16 @@ if(KOMPUTE_OPT_DISABLE_VK_DEBUG_LAYERS)
set(KOMPUTE_EXTRA_CXX_FLAGS "${KOMPUTE_EXTRA_CXX_FLAGS} -DKOMPUTE_DISABLE_VK_DEBUG_LAYERS=1")
endif()
if(KOMPUTE_OPT_DISABLE_GLSLANG)
set(KOMPUTE_EXTRA_CXX_FLAGS "${KOMPUTE_EXTRA_CXX_FLAGS} -DKOMPUTE_DISABLE_GLSLANG=1")
else()
if(KOMPUTE_OPT_INSTALL)
# Enable install parameters for glslang (overrides parameters passed)
set(ENABLE_GLSLANG_INSTALL ON CACHE BOOL "Enables install of glslang" FORCE)
set(BUILD_SHARED_LIBS ON CACHE BOOL "Enables build of shared libraries" FORCE)
endif()
endif()
set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} -DDEBUG=1 ${KOMPUTE_EXTRA_CXX_FLAGS} -DUSE_DEBUG_EXTENTIONS")
set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} -DRELEASE=1 ${KOMPUTE_EXTRA_CXX_FLAGS}")

View file

@ -70,7 +70,6 @@ mk_cmake:
-DKOMPUTE_OPT_ENABLE_SPDLOG=1 \
-DSPDLOG_INSTALL=1 \
-DKOMPUTE_OPT_CODE_COVERAGE=1 \
-DSHADERC_SKIP_TESTS=1 \
-G "Unix Makefiles"
mk_build_all:

View file

@ -1,5 +1,6 @@
#pragma once
#include "kompute/Core.hpp"
#include "kompute/ShaderUtil.hpp"
#include "kompute/shaders/shaderopmult.hpp"
#include "kompute/shaders/shaderlogisticregression.hpp"
#include "kompute/Manager.hpp"

View file

@ -107,6 +107,101 @@ extern py::object kp_debug, kp_info, kp_warning, kp_error;
#endif // KOMPUTE_SPDLOG_ENABLED
#endif // KOMPUTE_LOG_OVERRIDE
#include <iostream>
#include <vector>
#include <glslang/Public/ShaderLang.h>
#include <StandAlone/ResourceLimits.h>
#include <SPIRV/GlslangToSpv.h>
namespace kp {
static std::vector<char> spirv_from_sources(const std::vector<std::string>& sources,
const std::vector<std::string>& files = {},
const std::string& entryPoint = "main",
std::vector<std::pair<std::string,std::string>> definitions = {}) {
// Initialize glslang library.
glslang::InitializeProcess();
// Currently we don't support other shader types nor plan to
const EShLanguage language = EShLangCompute;
glslang::TShader shader(language);
std::vector<const char*> filesCStr(files.size()), sourcesCStr(sources.size());
for (size_t i = 0; i < sources.size(); i++) sourcesCStr[i] = sources[i].c_str();
if (files.size() > 1) {
assert(files.size() == sources.size());
for (size_t i = 0; i < files.size(); i++) filesCStr[i] = files[i].c_str();
shader.setStringsWithLengthsAndNames(sourcesCStr.data(), nullptr, filesCStr.data(), filesCStr.size());
}
else {
filesCStr = {""};
shader.setStringsWithLengthsAndNames(sourcesCStr.data(), nullptr, filesCStr.data(), sourcesCStr.size());
}
shader.setEntryPoint(entryPoint.c_str());
shader.setSourceEntryPoint(entryPoint.c_str());
std::string info_log = "";
const EShMessages messages = static_cast<EShMessages>(EShMsgDefault | EShMsgVulkanRules | EShMsgSpvRules);
if (!shader.parse(&glslang::DefaultTBuiltInResource, 100, false, messages))
{
info_log = std::string(shader.getInfoLog()) + "\n" + std::string(shader.getInfoDebugLog());
SPDLOG_ERROR(info_log);
throw std::runtime_error(info_log);
}
// Add shader to new program object.
glslang::TProgram program;
program.addShader(&shader);
// Link program.
if (!program.link(messages))
{
info_log = std::string(program.getInfoLog()) + "\n" + std::string(program.getInfoDebugLog());
SPDLOG_ERROR(info_log);
throw std::runtime_error(info_log);
}
// Save any info log that was generated.
if (shader.getInfoLog())
{
info_log += std::string(shader.getInfoLog()) + "\n" + std::string(shader.getInfoDebugLog()) + "\n";
SPDLOG_INFO(info_log);
}
glslang::TIntermediate *intermediate = program.getIntermediate(language);
// Translate to SPIRV.
if (!intermediate)
{
info_log += "Failed to get shared intermediate code.\n";
SPDLOG_ERROR(info_log);
throw std::runtime_error(info_log);
}
spv::SpvBuildLogger logger;
std::vector<std::uint32_t> spirv;
glslang::GlslangToSpv(*intermediate, spirv, &logger);
if (shader.getInfoLog())
{
info_log += logger.getAllMessages() + "\n";
SPDLOG_DEBUG(info_log);
}
// Shutdown glslang library.
glslang::FinalizeProcess();
return std::vector<char>((char*)spirv.data(), (char*)(spirv.data()+spirv.size()) );
}
static std::vector<char> spirv_from_source(const std::string& source) {
return spirv_from_sources({source});
}
}
/*
THIS FILE HAS BEEN AUTOMATICALLY GENERATED - DO NOT EDIT

View file

@ -105,6 +105,45 @@ if(KOMPUTE_OPT_BUILD_SINGLE_HEADER)
build_single_header)
endif()
#####################################################
#################### GLSLANG #######################
#####################################################
if(NOT KOMPUTE_OPT_DISABLE_GLSLANG)
if(KOMPUTE_OPT_REPO_SUBMODULE_BUILD)
add_subdirectory(${PROJECT_SOURCE_DIR}/external/glslang
${CMAKE_CURRENT_BINARY_DIR}/kompute_glslang)
target_include_directories(
kompute PRIVATE
${PROJECT_SOURCE_DIR}/external/glslang)
target_link_libraries(kompute
# Not including hlsl support
# HLSL
# glslang includes OGLCompiler, OSDependent, MachineIndependent
glslang
SPIRV
glslang-default-resource-limits)
else()
find_package(glslang CONFIG REQUIRED)
target_include_directories(
kompute PRIVATE
${GLSLANG_GENERATED_INCLUDEDIR})
target_link_libraries(kompute
# Not including hlsl support
# glslang::HLSL
# Adding explicit dependencies to match above
glslang
SPIRV
glslang-default-resource-limits)
endif()
endif()
add_library(kompute::kompute ALIAS kompute)
if(KOMPUTE_OPT_INSTALL)

View file

@ -1,13 +1,16 @@
#pragma once
#include <iostream>
#include <vector>
#include "kompute/Kompute.hpp"
#include "Core.hpp"
#include <glslang/Public/ShaderLang.h>
#include <StandAlone/ResourceLimits.h>
#include <SPIRV/GlslangToSpv.h>
namespace kp {
static std::vector<char> spirv_from_sources(const std::vector<std::string>& sources,
const std::vector<std::string>& files = {},
const std::string& entryPoint = "main",
@ -41,10 +44,10 @@ static std::vector<char> spirv_from_sources(const std::vector<std::string>& sour
if (!shader.parse(&glslang::DefaultTBuiltInResource, 100, false, messages))
{
info_log = std::string(shader.getInfoLog()) + "\n" + std::string(shader.getInfoDebugLog());
SPDLOG_ERROR(info_log);
throw std::runtime_error(info_log);
}
// Add shader to new program object.
glslang::TProgram program;
program.addShader(&shader);
@ -60,6 +63,7 @@ static std::vector<char> spirv_from_sources(const std::vector<std::string>& sour
if (shader.getInfoLog())
{
info_log += std::string(shader.getInfoLog()) + "\n" + std::string(shader.getInfoDebugLog()) + "\n";
SPDLOG_INFO(info_log);
}
glslang::TIntermediate *intermediate = program.getIntermediate(language);
@ -74,7 +78,12 @@ static std::vector<char> spirv_from_sources(const std::vector<std::string>& sour
spv::SpvBuildLogger logger;
std::vector<std::uint32_t> spirv;
glslang::GlslangToSpv(*intermediate, spirv, &logger);
info_log += logger.getAllMessages() + "\n";
if (shader.getInfoLog())
{
info_log += logger.getAllMessages() + "\n";
SPDLOG_DEBUG(info_log);
}
// Shutdown glslang library.
glslang::FinalizeProcess();
@ -82,7 +91,8 @@ static std::vector<char> spirv_from_sources(const std::vector<std::string>& sour
return std::vector<char>((char*)spirv.data(), (char*)(spirv.data()+spirv.size()) );
}
static std::vector<char> spirv_from_string(const std::string& source) {
static std::vector<char> spirv_from_source(const std::string& source) {
return spirv_from_sources({source});
}
}

View file

@ -38,45 +38,6 @@ target_link_libraries(test_kompute PRIVATE kompute)
add_test(NAME test_kompute COMMAND test_kompute)
#####################################################
#################### PYSHADERC #######################
#####################################################
set(SHADERC_SKIP_TESTS 1)
if(KOMPUTE_OPT_REPO_SUBMODULE_BUILD)
add_subdirectory(${PROJECT_SOURCE_DIR}/external/glslang
${CMAKE_CURRENT_BINARY_DIR}/kompute_glslang)
target_include_directories(
test_kompute PRIVATE
${PROJECT_SOURCE_DIR}/external/glslang)
target_link_libraries(test_kompute PRIVATE
# Not including hlsl support
# HLSL
# glslang includes OGLCompiler, OSDependent, MachineIndependent
glslang
SPIRV
glslang-default-resource-limits
MachineIndependent)
else()
find_package(glslang CONFIG REQUIRED)
target_include_directories(
test_kompute PRIVATE
${GLSLANG_GENERATED_INCLUDEDIR})
target_link_libraries(test_kompute PRIVATE
# Not including hlsl support
# glslang::HLSL
# Adding explicit dependencies to match above
glslang
SPIRV
glslang-default-resource-limits
MachineIndependent)
endif()
#####################################################
#################### CODECOV #######################
#####################################################

View file

@ -5,8 +5,6 @@
#include "kompute/Kompute.hpp"
#include "TestUtils.cpp"
TEST(TestAsyncOperations, TestManagerParallelExecution)
{
// This test is built for NVIDIA 1650. It assumes:
@ -57,7 +55,7 @@ TEST(TestAsyncOperations, TestManagerParallelExecution)
for (uint32_t i = 0; i < numParallel; i++) {
mgr.evalOpDefault<kp::OpAlgoBase>(
{ inputsSyncB[i] }, spirv_from_string(shader));
{ inputsSyncB[i] }, kp::spirv_from_source(shader));
}
auto endSync = std::chrono::high_resolution_clock::now();
@ -91,7 +89,7 @@ TEST(TestAsyncOperations, TestManagerParallelExecution)
mgrAsync.evalOpAsync<kp::OpAlgoBase>(
{ inputsAsyncB[i] },
"async" + std::to_string(i),
spirv_from_string(shader));
kp::spirv_from_source(shader));
}
for (uint32_t i = 0; i < numParallel; i++) {
@ -154,10 +152,10 @@ TEST(TestAsyncOperations, TestManagerAsyncExecution)
mgr.rebuild({ tensorA, tensorB });
mgr.evalOpAsync<kp::OpAlgoBase>(
{ tensorA }, "asyncOne", spirv_from_string(shader));
{ tensorA }, "asyncOne", kp::spirv_from_source(shader));
mgr.evalOpAsync<kp::OpAlgoBase>(
{ tensorB }, "asyncTwo", spirv_from_string(shader));
{ tensorB }, "asyncTwo", kp::spirv_from_source(shader));
mgr.evalOpAwait("asyncOne");
mgr.evalOpAwait("asyncTwo");

View file

@ -3,8 +3,6 @@
#include "kompute/Kompute.hpp"
#include "TestUtils.cpp"
TEST(TestDestroy, TestDestroyTensorSingle)
{
std::shared_ptr<kp::Tensor> tensorA{ new kp::Tensor({ 0, 0, 0 }) };
@ -30,7 +28,7 @@ TEST(TestDestroy, TestDestroyTensorSingle)
sq->begin();
sq->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq->end();
sq->eval();
@ -73,7 +71,7 @@ TEST(TestDestroy, TestDestroyTensorVector)
sq->begin();
sq->record<kp::OpAlgoBase>(
{ tensorA, tensorB }, spirv_from_string(shader));
{ tensorA, tensorB }, kp::spirv_from_source(shader));
sq->end();
sq->eval();
@ -138,7 +136,7 @@ TEST(TestDestroy, TestDestroySequenceSingle)
sq->begin();
sq->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq->end();
sq->eval();
@ -178,14 +176,14 @@ TEST(TestDestroy, TestDestroySequenceVector)
sq1 = mgr.sequence("One");
sq1->begin();
sq1->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq1->end();
sq1->eval();
sq2 = mgr.sequence("Two");
sq2->begin();
sq2->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq2->end();
sq2->eval();
@ -220,11 +218,11 @@ TEST(TestDestroy, TestDestroySequenceNameSingleInsideManager)
mgr.evalOp<kp::OpAlgoBase>(
{ tensorA }, "one",
spirv_from_string(shader));
kp::spirv_from_source(shader));
mgr.evalOp<kp::OpAlgoBase>(
{ tensorA }, "two",
spirv_from_string(shader));
kp::spirv_from_source(shader));
mgr.evalOpDefault<kp::OpTensorSyncLocal>({ tensorA });
@ -259,7 +257,7 @@ TEST(TestDestroy, TestDestroySequenceNameSingleOutsideManager)
sq1 = mgr.sequence("One");
sq1->begin();
sq1->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq1->end();
sq1->eval();
@ -293,11 +291,11 @@ TEST(TestDestroy, TestDestroySequenceNameVectorInsideManager)
mgr.evalOp<kp::OpAlgoBase>(
{ tensorA }, "one",
spirv_from_string(shader));
kp::spirv_from_source(shader));
mgr.evalOp<kp::OpAlgoBase>(
{ tensorA }, "two",
spirv_from_string(shader));
kp::spirv_from_source(shader));
mgr.evalOpDefault<kp::OpTensorSyncLocal>({ tensorA });
@ -327,11 +325,11 @@ TEST(TestDestroy, TestDestroySequenceNameVectorOutsideManager)
mgr.evalOp<kp::OpAlgoBase>(
{ tensorA }, "one",
spirv_from_string(shader));
kp::spirv_from_source(shader));
mgr.evalOp<kp::OpAlgoBase>(
{ tensorA }, "two",
spirv_from_string(shader));
kp::spirv_from_source(shader));
mgr.evalOpDefault<kp::OpTensorSyncLocal>({ tensorA });
@ -361,7 +359,7 @@ TEST(TestDestroy, TestDestroySequenceNameDefaultOutsideManager)
mgr.evalOpDefault<kp::OpAlgoBase>(
{ tensorA },
spirv_from_string(shader));
kp::spirv_from_source(shader));
mgr.evalOpDefault<kp::OpTensorSyncLocal>({ tensorA });

View file

@ -3,8 +3,6 @@
#include "kompute/Kompute.hpp"
#include "TestUtils.cpp"
TEST(TestMultipleAlgoExecutions, SingleSequenceRecord)
{
@ -30,11 +28,11 @@ TEST(TestMultipleAlgoExecutions, SingleSequenceRecord)
sq->begin();
sq->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq->record<kp::OpTensorSyncLocal>({ tensorA });
@ -75,19 +73,19 @@ TEST(TestMultipleAlgoExecutions, MultipleCmdBufRecords)
// Then perform the computations
sq->begin();
sq->record<kp::OpAlgoBase>({ tensorA },
spirv_from_string(shader));
kp::spirv_from_source(shader));
sq->end();
sq->eval();
sq->begin();
sq->record<kp::OpAlgoBase>({ tensorA },
spirv_from_string(shader));
kp::spirv_from_source(shader));
sq->end();
sq->eval();
sq->begin();
sq->record<kp::OpAlgoBase>({ tensorA },
spirv_from_string(shader));
kp::spirv_from_source(shader));
sq->end();
sq->eval();
@ -124,7 +122,7 @@ TEST(TestMultipleAlgoExecutions, MultipleSequences)
sq->begin();
sq->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq->end();
sq->eval();
@ -137,7 +135,7 @@ TEST(TestMultipleAlgoExecutions, MultipleSequences)
sq->begin();
sq->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq->end();
sq->eval();
@ -150,7 +148,7 @@ TEST(TestMultipleAlgoExecutions, MultipleSequences)
sq->begin();
sq->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq->end();
sq->eval();
@ -208,7 +206,7 @@ TEST(TestMultipleAlgoExecutions, SingleRecordMultipleEval)
sq->begin();
sq->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq->end();
@ -267,7 +265,7 @@ TEST(TestMultipleAlgoExecutions, ManagerEvalMultSourceStrOpCreate)
mgr.evalOpDefault<kp::OpAlgoBase>(
{ tensorInA, tensorInB, tensorOut },
spirv_from_string(shader));
kp::spirv_from_source(shader));
mgr.evalOpDefault<kp::OpTensorSyncLocal>({ tensorOut });
@ -310,7 +308,7 @@ TEST(TestMultipleAlgoExecutions, ManagerEvalMultSourceStrMgrCreate)
mgr.evalOpDefault<kp::OpAlgoBase>(
{ tensorInA, tensorInB, tensorOut },
spirv_from_string(shader));
kp::spirv_from_source(shader));
mgr.evalOpDefault<kp::OpTensorSyncLocal>({ tensorOut });
@ -342,7 +340,7 @@ TEST(TestMultipleAlgoExecutions, SequenceAlgoDestroyOutsideManagerScope)
sq->begin();
sq->record<kp::OpAlgoBase>(
{ tensorA }, spirv_from_string(shader));
{ tensorA }, kp::spirv_from_source(shader));
sq->end();
sq->eval();

View file

@ -3,8 +3,6 @@
#include "kompute/Kompute.hpp"
#include "TestUtils.cpp"
TEST(TestProcessingIterations, IterateThroughMultipleSumAndCopies)
{
kp::Manager mgr;
@ -55,7 +53,7 @@ TEST(TestProcessingIterations, IterateThroughMultipleSumAndCopies)
sq->record<kp::OpAlgoBase>(
{ tensorA, tensorB },
spirv_from_string(shader));
kp::spirv_from_source(shader));
sq->record<kp::OpTensorCopy>({ tensorB, tensorA });
sq->end();

View file

@ -5,8 +5,6 @@
#include "kompute_test/shaders/shadertest_op_custom_shader.hpp"
#include "TestUtils.cpp"
TEST(TestOpAlgoBase, ShaderRawDataFromConstructor)
{
kp::Manager mgr;
@ -31,7 +29,7 @@ TEST(TestOpAlgoBase, ShaderRawDataFromConstructor)
)");
mgr.evalOpDefault<kp::OpAlgoBase>(
{ tensorA, tensorB }, spirv_from_string(shader));
{ tensorA, tensorB }, kp::spirv_from_source(shader));
mgr.evalOpDefault<kp::OpTensorSyncLocal>({ tensorA, tensorB });

View file

@ -1,10 +1,7 @@
#include "gtest/gtest.h"
#include "kompute/Kompute.hpp"
#include "TestUtils.cpp"
TEST(TestSpecializationConstants, TestTwoConstants)
{
std::shared_ptr<kp::Tensor> tensorA{ new kp::Tensor({ 0, 0, 0 }) };
@ -38,7 +35,7 @@ TEST(TestSpecializationConstants, TestTwoConstants)
sq->begin();
sq->record<kp::OpAlgoBase>(
{ tensorA, tensorB },
spirv_from_string(shader),
kp::spirv_from_source(shader),
kp::Workgroup(), spec);
sq->end();