diff options
| author | Yong He <yonghe@outlook.com> | 2024-08-18 21:57:24 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-08-18 21:57:24 -0700 |
| commit | ecf85df6eee3da76ef54b14e4ab083f22da89e46 (patch) | |
| tree | 4656f9c11a1f7f40550d469fecbcd7a16c541f52 /source/slang/slang-legalize-types.cpp | |
| parent | ca5d303748517889a5d5849224671fa8945e1c6d (diff) | |
Variadic Generics Part 2: IR lowering and specialization. (#4849)
* Variadic Generics Part 2: IR lowering and specialization.
* Update design doc status.
* Update design doc.
* Resolve review comments.
Diffstat (limited to 'source/slang/slang-legalize-types.cpp')
| -rw-r--r-- | source/slang/slang-legalize-types.cpp | 19 |
1 files changed, 14 insertions, 5 deletions
diff --git a/source/slang/slang-legalize-types.cpp b/source/slang/slang-legalize-types.cpp index 5e8390cef..6009ef33e 100644 --- a/source/slang/slang-legalize-types.cpp +++ b/source/slang/slang-legalize-types.cpp @@ -287,6 +287,11 @@ struct TupleTypeBuilder { specialType = legalFieldType; } + + // `void` is currently legalized to simple, but we don't want to add a + // `void` field to the struct. + if (legalLeafType.getSimple()->getOp() == kIROp_VoidType) + return; } break; @@ -419,7 +424,6 @@ struct TupleTypeBuilder bool isSpecialField = context->isSpecialType(fieldType); auto legalFieldType = legalizeType(context, fieldType); - addField( field->getKey(), legalFieldType, @@ -1385,10 +1389,15 @@ LegalType legalizeTypeImpl( context, arrayType->getElementType()); - // If element type hasn't change, return original type. - if (legalElementType.flavor == LegalType::Flavor::simple && - legalElementType.getSimple() == arrayType->getElementType()) - return LegalType::simple(arrayType); + if (legalElementType.flavor == LegalType::Flavor::simple) + { + if (legalElementType.getSimple()->getOp() == kIROp_VoidType) + return LegalType(); + + // If element type hasn't change, return original type. + if (legalElementType.getSimple() == arrayType->getElementType()) + return LegalType::simple(arrayType); + } ArrayLegalTypeWrapper wrapper; wrapper.arrayType = arrayType; |
