diff options
Diffstat (limited to 'source')
| -rw-r--r-- | source/slang/slang-emit-cpp.cpp | 56 | ||||
| -rw-r--r-- | source/slang/slang-ir-legalize-types.cpp | 9 | ||||
| -rw-r--r-- | source/slang/slang-ir-lower-generic-function.cpp | 3 | ||||
| -rw-r--r-- | source/slang/slang-ir-lower-generic-type.cpp | 9 |
4 files changed, 54 insertions, 23 deletions
diff --git a/source/slang/slang-emit-cpp.cpp b/source/slang/slang-emit-cpp.cpp index a178dfe67..d6764ae06 100644 --- a/source/slang/slang-emit-cpp.cpp +++ b/source/slang/slang-emit-cpp.cpp @@ -1270,13 +1270,26 @@ bool CPPSourceEmitter::tryEmitInstExprImpl(IRInst* inst, const EmitOpInfo& inOut IRInst* baseInst = getElementInst->getBase(); IRType* baseType = baseInst->getDataType(); - if (as<IRVectorType>(baseType)) + if (auto vectorBaseType = as<IRVectorType>(baseType)) { - m_writer->emit("_slang_vector_get_element("); - emitOperand(baseInst, getInfo(EmitOp::General)); - m_writer->emit(", "); - emitOperand(getElementInst->getIndex(), getInfo(EmitOp::General)); - m_writer->emit(")"); + if (auto intLitIndex = as<IRIntLit>(getElementInst->getIndex())) + { + // For static index, we can emit simpler code using the `.x`, `.y` members. + auto outerPrec = getInfo(EmitOp::General); + auto prec = getInfo(EmitOp::Postfix); + emitOperand(baseInst, leftSide(outerPrec, prec)); + m_writer->emit("."); + m_writer->emit(getVectorElementNames(vectorBaseType)[intLitIndex->getValue()]); + } + else + { + // For dynamic index, we emit using `_slang_vector_get_element` intrinsics. + m_writer->emit("_slang_vector_get_element("); + emitOperand(baseInst, getInfo(EmitOp::General)); + m_writer->emit(", "); + emitOperand(getElementInst->getIndex(), getInfo(EmitOp::General)); + m_writer->emit(")"); + } return true; } else if (as<IRMatrixType>(baseType)) @@ -1297,24 +1310,37 @@ bool CPPSourceEmitter::tryEmitInstExprImpl(IRInst* inst, const EmitOpInfo& inOut IRInst* baseInst = getElementInst->getBase(); IRType* baseType = as<IRPtrTypeBase>(baseInst->getDataType())->getValueType(); - if (as<IRVectorType>(baseType)) + if (auto vectorBaseType = as<IRVectorType>(baseType)) { - m_writer->emit("_slang_vector_get_element_ptr("); - emitOperand(baseInst, getInfo(EmitOp::General)); - m_writer->emit(", "); - emitOperand(getElementInst->getIndex(), getInfo(EmitOp::General)); - m_writer->emit(")"); + if (auto intLitIndex = as<IRIntLit>(getElementInst->getIndex())) + { + // For static index, we can emit simpler code using the `.x`, `.y` members. + m_writer->emit("&("); + auto outerPrec = getInfo(EmitOp::General); + auto prec = getInfo(EmitOp::Postfix); + emitOperand(baseInst, leftSide(outerPrec, prec)); + m_writer->emit("->"); + m_writer->emit(getVectorElementNames(vectorBaseType)[intLitIndex->getValue()]); + m_writer->emit(")"); + } + else + { + m_writer->emit("_slang_vector_get_element_ptr("); + emitOperand(baseInst, getInfo(EmitOp::General)); + m_writer->emit(", "); + emitOperand(getElementInst->getIndex(), getInfo(EmitOp::General)); + m_writer->emit(")"); + } return true; } else if (as<IRMatrixType>(baseType)) { - m_writer->emit("&("); + m_writer->emit("("); auto outerPrec = getInfo(EmitOp::General); auto prec = getInfo(EmitOp::Postfix); emitOperand(baseInst, leftSide(outerPrec, prec)); - m_writer->emit("->rows["); + m_writer->emit("->rows + "); emitOperand(getElementInst->getIndex(), getInfo(EmitOp::General)); - m_writer->emit("]"); m_writer->emit(")"); return true; } diff --git a/source/slang/slang-ir-legalize-types.cpp b/source/slang/slang-ir-legalize-types.cpp index b9d494c21..52b5bc72f 100644 --- a/source/slang/slang-ir-legalize-types.cpp +++ b/source/slang/slang-ir-legalize-types.cpp @@ -2159,6 +2159,8 @@ static LegalVal legalizeInst( if (newArgs[aa] != inst->getOperand(aa)) recreate = true; } + if (inst->getFullType() != legalType.getSimple()) + recreate = true; if (recreate) { IRBuilder builder(inst->getModule()); @@ -2169,8 +2171,6 @@ static LegalVal legalizeInst( context->replacedInstructions.add(inst); return LegalVal::simple(newInst); } - if (inst->getFullType() != legalType.getSimple()) - inst->setFullType(legalType.getSimple()); return LegalVal::simple(inst); } @@ -3641,7 +3641,10 @@ struct IRTypeLegalizationPass // // * `i` is a user of `inst`, or // * `i` is a child of `inst`. - // + // + if (legalVal.flavor == LegalVal::Flavor::simple) + inst = legalVal.irValue; + for( auto use = inst->firstUse; use; use = use->nextUse ) { auto user = use->getUser(); diff --git a/source/slang/slang-ir-lower-generic-function.cpp b/source/slang/slang-ir-lower-generic-function.cpp index ad43aff95..e45b20563 100644 --- a/source/slang/slang-ir-lower-generic-function.cpp +++ b/source/slang/slang-ir-lower-generic-function.cpp @@ -322,7 +322,8 @@ namespace Slang return; auto interfaceType = maybeLowerInterfaceType(cast<IRInterfaceType>(witnessTableType->getConformanceType())); interfaceRequirementVal = sharedContext->findInterfaceRequirementVal(interfaceType, lookupInst->getRequirementKey()); - lookupInst->setFullType((IRType*)interfaceRequirementVal); + IRBuilder builder(lookupInst); + builder.replaceOperand(&lookupInst->typeUse, interfaceRequirementVal); } void lowerSpecialize(IRSpecialize* specializeInst) diff --git a/source/slang/slang-ir-lower-generic-type.cpp b/source/slang/slang-ir-lower-generic-type.cpp index 398db4f78..256978346 100644 --- a/source/slang/slang-ir-lower-generic-type.cpp +++ b/source/slang/slang-ir-lower-generic-type.cpp @@ -14,7 +14,7 @@ namespace Slang { SharedGenericsLoweringContext* sharedContext; - void processInst(IRInst* inst) + IRInst* processInst(IRInst* inst) { // Ensure public struct types has RTTI object defined. if (as<IRStructType>(inst)) @@ -27,7 +27,7 @@ namespace Slang // Don't modify type insts themselves. if (as<IRType>(inst)) - return; + return inst; IRBuilder builderStorage(sharedContext->module); auto builder = &builderStorage; @@ -35,7 +35,7 @@ namespace Slang auto newType = sharedContext->lowerType(builder, inst->getFullType()); if (newType != inst->getFullType()) - inst->setFullType((IRType*)newType); + inst = builder->replaceOperand(&inst->typeUse, newType); switch (inst->getOp()) { @@ -51,6 +51,7 @@ namespace Slang } break; } + return inst; } void processModule() @@ -64,7 +65,7 @@ namespace Slang sharedContext->workList.removeLast(); sharedContext->workListSet.Remove(inst); - processInst(inst); + inst = processInst(inst); for (auto child = inst->getLastChild(); child; child = child->getPrevInst()) { |
