Skip to content

Commit

Permalink
VideoCommon: Remove old iLog field
Browse files Browse the repository at this point in the history
Prior to 7854bd7, this was used by the debugger for the OpenGL and D3D9 plugins to control logging (via PRIM_LOG and INFO_LOG/DEBUG_LOG in VideoCommon code; PRIM_LOG was changed in 77215fd), and also framedumping (removed in 64927a2 and 2d8515c), shader dumping (removed in 2d8515c and this commit), and texture dumping (removed in 54aeec7). Apart from shader dumping, all of these features have modern alternatives, and shader source code can be seen in RenderDoc if "Enable API Validation Layers" is checked (which also enables source attachment), so there's no point in keeping this around.
  • Loading branch information
Pokechu22 committed Aug 4, 2022
1 parent ceae42b commit 354530c
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 32 deletions.
25 changes: 0 additions & 25 deletions Source/Core/VideoCommon/Spirv.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -155,31 +155,6 @@ CompileShaderToSPV(EShLanguage stage, APIType api_type,
if (!spv_messages.empty())
WARN_LOG_FMT(VIDEO, "SPIR-V conversion messages: {}", spv_messages);

// Dump source code of shaders out to file if enabled.
if (g_ActiveConfig.iLog & CONF_SAVESHADERS)
{
static int counter = 0;
std::string filename = StringFromFormat("%s%s_%04i.txt", File::GetUserPath(D_DUMP_IDX).c_str(),
stage_filename, counter++);

std::ofstream stream;
File::OpenFStream(stream, filename, std::ios_base::out);
if (stream.good())
{
stream << source << std::endl;
stream << "Shader Info Log:" << std::endl;
stream << shader->getInfoLog() << std::endl;
stream << shader->getInfoDebugLog() << std::endl;
stream << "Program Info Log:" << std::endl;
stream << program->getInfoLog() << std::endl;
stream << program->getInfoDebugLog() << std::endl;
stream << "SPIR-V conversion messages: " << std::endl;
stream << spv_messages;
stream << "SPIR-V:" << std::endl;
spv::Disassemble(stream, out_code);
}
}

return out_code;
}
} // namespace
Expand Down
7 changes: 0 additions & 7 deletions Source/Core/VideoCommon/VideoConfig.h
Original file line number Diff line number Diff line change
Expand Up @@ -17,12 +17,6 @@
#include "VideoCommon/GraphicsModSystem/Config/GraphicsModGroup.h"
#include "VideoCommon/VideoCommon.h"

// Log in two categories, and save three other options in the same byte
#define CONF_LOG 1
#define CONF_PRIMLOG 2
#define CONF_SAVETARGETS 8
#define CONF_SAVESHADERS 16

constexpr int EFB_SCALE_AUTO_INTEGRAL = 0;

enum class AspectMode : int
Expand Down Expand Up @@ -138,7 +132,6 @@ struct VideoConfig final
bool bFastDepthCalc = false;
bool bVertexRounding = false;
int iEFBAccessTileSize = 0;
int iLog = 0; // CONF_ bits
int iSaveTargetId = 0; // TODO: Should be dropped
u32 iMissingColorValue = 0;
bool bFastTextureSampling = false;
Expand Down

0 comments on commit 354530c

Please sign in to comment.