Hash :
af0386cb
Author :
Date :
2025-08-27T10:44:53
Reorder structs entirely made of samplers to the end of struct An earlier change [1] moved samplers to the end of structs so that extracting them does not disturb the indices to the non-sampler fields. This change does the same for structs that only contain samplers, because they too get removed from the parent struct, disturbing indices. [1]:https://chromium-review.googlesource.com/c/angle/angle/+/5748789 Bug: chromium:438038775 Change-Id: I7ac2ea13723bf06b4b172b6e70f0dbe85bc45ce3 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6891071 Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org>
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207
//
// Copyright 2025 The ANGLE Project Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
//
#include "compiler/translator/tree_ops/GatherDefaultUniforms.h"
#include "GLSLANG/ShaderVars.h"
#include "angle_gl.h"
#include "common/debug.h"
#include "compiler/translator/Compiler.h"
#include "compiler/translator/ImmutableString.h"
#include "compiler/translator/SymbolUniqueId.h"
#include "compiler/translator/tree_util/IntermNode_util.h"
#include "compiler/translator/tree_util/IntermTraverse.h"
#include "compiler/translator/tree_util/ReplaceVariable.h"
#include "compiler/translator/util.h"
namespace sh
{
namespace
{
class ReplaceDefaultUniformsTraverser : public TIntermTraverser
{
public:
ReplaceDefaultUniformsTraverser(const VariableReplacementMap &variableMap)
: TIntermTraverser(true, false, false), mVariableMap(variableMap)
{}
bool visitDeclaration(Visit visit, TIntermDeclaration *node) override
{
const TIntermSequence &sequence = *(node->getSequence());
TIntermTyped *variable = sequence.front()->getAsTyped();
const TType &type = variable->getType();
if (IsDefaultUniform(type))
{
// Remove the uniform declaration.
TIntermSequence emptyReplacement;
mMultiReplacements.emplace_back(getParentNode()->getAsBlock(), node,
std::move(emptyReplacement));
return false;
}
return true;
}
void visitSymbol(TIntermSymbol *symbol) override
{
const TVariable &variable = symbol->variable();
const TType &type = variable.getType();
if (!IsDefaultUniform(type) || gl::IsBuiltInName(variable.name().data()))
{
return;
}
ASSERT(mVariableMap.count(&variable) > 0);
queueReplacement(mVariableMap.at(&variable)->deepCopy(), OriginalNode::IS_DROPPED);
}
private:
const VariableReplacementMap &mVariableMap;
};
// Fields of nameless interface blocks are TVariables. So when creating a new interface blocks we
// need to create all the TVariables representing its fields.
TIntermSymbol *CreateVariableForFieldOfNamelessInterfaceBlock(const TVariable *object,
int index,
TSymbolTable *symbolTable)
{
ASSERT(object->getType().getInterfaceBlock());
const TFieldList &fieldList = object->getType().getInterfaceBlock()->fields();
ASSERT(0 <= index);
ASSERT(static_cast<size_t>(index) < fieldList.size());
ASSERT(object->symbolType() == SymbolType::Empty);
TType *replacementType = new TType(*fieldList[index]->type());
replacementType->setInterfaceBlockField(object->getType().getInterfaceBlock(), index);
TVariable *replacementVariable = new TVariable(symbolTable, fieldList[index]->name(),
replacementType, fieldList[index]->symbolType());
return new TIntermSymbol(replacementVariable);
}
} // namespace
bool IsDefaultUniform(const TType &type)
{
return type.getQualifier() == EvqUniform && type.getInterfaceBlock() == nullptr &&
!IsOpaqueType(type.getBasicType());
}
TSet<ImmutableString> *GetActiveUniforms(const std::vector<ShaderVariable> &defaultUniforms)
{
auto activeUniforms = new TSet<ImmutableString>();
for (const ShaderVariable &uniform : defaultUniforms)
{
if (uniform.active)
{
activeUniforms->insert(uniform.name);
}
}
return activeUniforms;
}
bool GatherDefaultUniforms(TCompiler *compiler,
TIntermBlock *root,
TSymbolTable *symbolTable,
gl::ShaderType shaderType,
const ImmutableString &uniformBlockType,
const ImmutableString &uniformBlockVarName,
const TVariable **outUniformBlock)
{
// First, collect all default uniforms and declare a uniform block.
TFieldList *uniformList = new TFieldList;
TVector<const TVariable *> uniformVars;
TSet<ImmutableString> *activeUniforms = GetActiveUniforms(compiler->getUniforms());
for (TIntermNode *node : *root->getSequence())
{
TIntermDeclaration *decl = node->getAsDeclarationNode();
if (decl == nullptr)
{
continue;
}
const TIntermSequence &sequence = *(decl->getSequence());
TIntermSymbol *symbol = sequence.front()->getAsSymbolNode();
if (symbol == nullptr)
{
continue;
}
const TType &type = symbol->getType();
// Only gather active default uniforms.
if (IsDefaultUniform(type) && activeUniforms->count(symbol->getName()) != 0)
{
TType *fieldType = new TType(type);
uniformList->push_back(new TField(fieldType, symbol->getName(), symbol->getLine(),
symbol->variable().symbolType()));
uniformVars.push_back(&symbol->variable());
}
}
VariableReplacementMap variableMap;
if (uniformList->empty())
{
*outUniformBlock = nullptr;
}
else
{
TLayoutQualifier layoutQualifier = TLayoutQualifier::Create();
layoutQualifier.blockStorage = EbsStd140;
TInterfaceBlock *interfaceBlock =
DeclareInterfaceBlock(symbolTable, uniformList, layoutQualifier, uniformBlockType);
// Set the mIsDefaultUniformBlock bit because the interfaceBlock represents default uniform
// interfaceBlock.
// Later when traversing the AST and output SPIRV, we will rely on this bit to decide if we
// want to transform FP32 to FP16 for float based on if the float vars are inside the
// default uniform block.
interfaceBlock->setDefaultUniformBlock();
*outUniformBlock = DeclareInterfaceBlockVariable(
root, symbolTable, EvqUniform, interfaceBlock, layoutQualifier,
TMemoryQualifier::Create(), 0, uniformBlockVarName);
// Create a map from the uniform variables to new variables that reference the fields of the
// block.
for (size_t fieldIndex = 0; fieldIndex < uniformVars.size(); ++fieldIndex)
{
const TVariable *variable = uniformVars[fieldIndex];
if ((*outUniformBlock)->symbolType() == SymbolType::Empty)
{
variableMap[variable] = CreateVariableForFieldOfNamelessInterfaceBlock(
*outUniformBlock, static_cast<int>(fieldIndex), symbolTable);
}
else
{
variableMap[variable] = AccessFieldOfNamedInterfaceBlock(
*outUniformBlock, static_cast<int>(fieldIndex));
}
}
}
// Finally transform the AST and make sure references to the uniforms are replaced with the new
// variables.
ReplaceDefaultUniformsTraverser defaultTraverser(variableMap);
root->traverse(&defaultTraverser);
return defaultTraverser.updateTree(compiler, root);
}
} // namespace sh