diff options
Diffstat (limited to 'source')
| -rw-r--r-- | source/slang/ir-legalize-types.cpp | 14 | ||||
| -rw-r--r-- | source/slang/legalize-types.cpp | 17 |
2 files changed, 30 insertions, 1 deletions
diff --git a/source/slang/ir-legalize-types.cpp b/source/slang/ir-legalize-types.cpp index df3e190e1..9136ad671 100644 --- a/source/slang/ir-legalize-types.cpp +++ b/source/slang/ir-legalize-types.cpp @@ -21,6 +21,8 @@ namespace Slang LegalVal LegalVal::tuple(RefPtr<TuplePseudoVal> tupleVal) { + SLANG_ASSERT(tupleVal->elements.Count()); + LegalVal result; result.flavor = LegalVal::Flavor::tuple; result.obj = tupleVal; @@ -343,6 +345,9 @@ static LegalVal legalizeFieldAddress( switch (legalPtrOperand.flavor) { + case LegalVal::Flavor::none: + return LegalVal(); + case LegalVal::Flavor::simple: return LegalVal::simple( builder->emitFieldAddress( @@ -457,6 +462,9 @@ static LegalVal legalizeFieldExtract( switch (legalStructOperand.flavor) { + case LegalVal::Flavor::none: + return LegalVal(); + case LegalVal::Flavor::simple: return LegalVal::simple( builder->emitFieldExtract( @@ -571,6 +579,9 @@ static LegalVal legalizeGetElementPtr( switch (legalPtrOperand.flavor) { + case LegalVal::Flavor::none: + return LegalVal(); + case LegalVal::Flavor::simple: return LegalVal::simple( builder->emitElementAddress( @@ -679,6 +690,9 @@ static LegalVal legalizeMakeStruct( switch(legalType.flavor) { + case LegalType::Flavor::none: + return LegalVal(); + case LegalType::Flavor::simple: { List<IRInst*> args; diff --git a/source/slang/legalize-types.cpp b/source/slang/legalize-types.cpp index 6ab597d93..1f0fe93cb 100644 --- a/source/slang/legalize-types.cpp +++ b/source/slang/legalize-types.cpp @@ -22,6 +22,8 @@ LegalType LegalType::implicitDeref( LegalType LegalType::tuple( RefPtr<TuplePseudoType> tupleType) { + SLANG_ASSERT(tupleType->elements.Count()); + LegalType result; result.flavor = Flavor::tuple; result.obj = tupleType; @@ -173,6 +175,10 @@ struct TupleTypeBuilder } break; + case LegalType::Flavor::none: + anyComplex = true; + break; + case LegalType::Flavor::implicitDeref: { // TODO: we may want to say that any use @@ -305,7 +311,7 @@ struct TupleTypeBuilder // This helps get rid of emtpy structs that often trips up the // downstream compiler if (!anyOrdinary && !anySpecial && !anyComplex) - return LegalType::tuple(new TuplePseudoType()); + return LegalType(); // If we didn't see anything "special" // then we can use the type as-is. @@ -437,6 +443,9 @@ static LegalType createLegalUniformBufferType( { switch (legalElementType.flavor) { + case LegalType::Flavor::none: + return LegalType(); + case LegalType::Flavor::simple: { // Easy case: we just have a simple element type, @@ -540,6 +549,9 @@ static LegalType createLegalPtrType( { switch (legalValueType.flavor) { + case LegalType::Flavor::none: + return LegalType(); + case LegalType::Flavor::simple: { // Easy case: we just have a simple element type, @@ -675,6 +687,9 @@ static LegalType wrapLegalType( { switch (legalType.flavor) { + case LegalType::Flavor::none: + return LegalType(); + case LegalType::Flavor::simple: { return ordinaryWrapper->wrap(context, legalType.getSimple()); |
