From b5a4161a801a573179b1f552e5c53748d2667b03 Mon Sep 17 00:00:00 2001 From: jsmall-nvidia Date: Wed, 19 Aug 2020 15:51:43 -0400 Subject: Remove IncludeHandler. (#1505) nvAPI -> NVAPI nvAPIPath -> nvapiPath DxcIncludeHandler don't reference count. nv-api-path -> nvapi-path Co-authored-by: Tim Foley --- source/slang/slang-check-decl.cpp | 19 ------------------- 1 file changed, 19 deletions(-) (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 13b818b26..10e3818c4 100644 --- a/source/slang/slang-check-decl.cpp +++ b/source/slang/slang-check-decl.cpp @@ -3520,25 +3520,6 @@ namespace Slang return subst; } -#if 0 - // For simplicity we will make having a definition of a function include having a body or a target intrinsics defined. - // It may be useful to add other modifiers to mark as having body - for example perhaps - // any target intrinsic modifier (like SPIR-V version) should be included. - // - // Note that not having this check around TargetIntrinsicModifier can lead to a crash in the compiler - // with a definition, followed by a declaration with a target intrinsic. - // That this doesn't appear to be the case with other modifiers. - // TODO: - // We may want to be able to add target intrinsics with other declarations, that being the case this logic - // would need to change. - // We might also want are more precise error that pointed out the actually problem - because strictly speaking - // having a target intrinsic isn't a 'body'. - bool _isDefinition(FuncDecl* decl) - { - return decl->body || decl->hasModifier(); - } -#endif - typedef Dictionary TargetDeclDictionary; static void _addTargetModifiers(CallableDecl* decl, TargetDeclDictionary& ioDict) -- cgit v1.2.3