From 387f2be1e48a805ef0da34510a5ae0ebc0ba9c3e Mon Sep 17 00:00:00 2001 From: Yong He Date: Wed, 15 Jan 2025 20:43:01 -0800 Subject: Inline global constants that contains opaque handles for legalization. (#6098) * Inline global constants that contains opaque handles for legalization. * Add diagnostics on opaque type global variables. * Fix. * Fix test. --- source/slang/slang-check-decl.cpp | 12 ++++++++++++ 1 file changed, 12 insertions(+) (limited to 'source/slang/slang-check-decl.cpp') diff --git a/source/slang/slang-check-decl.cpp b/source/slang/slang-check-decl.cpp index 04d5b7a75..2a5b5a4e3 100644 --- a/source/slang/slang-check-decl.cpp +++ b/source/slang/slang-check-decl.cpp @@ -2392,6 +2392,18 @@ void SemanticsDeclBodyVisitor::checkVarDeclCommon(VarDeclBase* varDecl) { getSink()->diagnose(varDecl, Diagnostics::varCannotBeUnsized); } + + bool isOpaque = (((int)varTypeTags & (int)TypeTag::Opaque) != 0); + if (isOpaque && isGlobalDecl(varDecl) && !varDecl->hasModifier() && + varDecl->hasModifier()) + { + // Opaque type global variable must be const. + getSink()->diagnose(varDecl, Diagnostics::globalVarCannotHaveOpaqueType); + if (varDecl->initExpr) + getSink()->diagnose(varDecl, Diagnostics::doYouMeanStaticConst); + else + getSink()->diagnose(varDecl, Diagnostics::doYouMeanUniform); + } } if (auto elementType = getConstantBufferElementType(varDecl->getType())) -- cgit v1.2.3