encounter
/
dawn-cmake
mirror of https://github.com/encounter/dawn-cmake.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

tint: Fix Matrix::FriendlyName() for inferred type 

The type field may be nullptr when a matrix constructor is used with
an inferred element type.

Change-Id: Ic11ba6a8efbed2a641c8a5ad3bbf0d074e4ca973
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/87280
Reviewed-by: Ben Clayton <bclayton@google.com>
Commit-Queue: James Price <jrprice@google.com>
Auto-Submit: James Price <jrprice@google.com>
This commit is contained in:
James Price 2022-04-20 01:30:25 +00:00 committed by Dawn LUCI CQ
parent 13e9699b1c
commit b6f4c59845
2 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/tint/ast/matrix.cc
View File

@ -39,8 +39,10 @@ Matrix::~Matrix() = default;
std::string Matrix::FriendlyName(const SymbolTable& symbols) const {
std::ostringstream out;
out << "mat" << columns << "x" << rows << "<" << type->FriendlyName(symbols)
<< ">";
out << "mat" << columns << "x" << rows;
if (type) {
out << "<" << type->FriendlyName(symbols) << ">";
}
return out.str();
}

5
src/tint/ast/matrix_test.cc
View File

@ -46,5 +46,10 @@ TEST_F(AstMatrixTest, FriendlyName) {
EXPECT_EQ(m->FriendlyName(Symbols()), "mat2x3<i32>");
}
TEST_F(AstMatrixTest, FriendlyName_WithoutType) {
auto* m = create<Matrix>(nullptr, 3, 2);
EXPECT_EQ(m->FriendlyName(Symbols()), "mat2x3");
}
} // namespace
} // namespace tint::ast