diff options
| author | Tim Foley <tfoley@nvidia.com> | 2017-10-30 10:03:52 -0700 |
|---|---|---|
| committer | Tim Foley <tfoley@nvidia.com> | 2017-10-30 10:26:22 -0700 |
| commit | 11f44241ffef478560eaba79af330c16f0bc8d69 (patch) | |
| tree | 5f7b06149f5d6e0af9241db8f60b28b9c07785f1 /source/slang/check.cpp | |
| parent | 42f1cff5c1471e6bc3988a9810c20b8bcc1c84dd (diff) | |
Allow for implicit `this` expressions.
- When peforming ordinary lookup, if the container declaration for a scope is an aggregate type or `extension` decl, then use a "breadcrumb" to make sure that we use a `this` expression as the base of any resulting declaration reference
- Add a test case for implicit `this` usage
- Update constrained generic test case to use implicit `this` for member reference, as was originally intended
Diffstat (limited to 'source/slang/check.cpp')
| -rw-r--r-- | source/slang/check.cpp | 34 |
1 files changed, 34 insertions, 0 deletions
diff --git a/source/slang/check.cpp b/source/slang/check.cpp index 9b707d218..0c35c4bf4 100644 --- a/source/slang/check.cpp +++ b/source/slang/check.cpp @@ -196,6 +196,16 @@ namespace Slang return derefExpr; } + RefPtr<Expr> createImplicitThisMemberExpr( + Type* type, + SourceLoc loc) + { + RefPtr<ThisExpr> expr = new ThisExpr(); + expr->type = type; + expr->loc = loc; + return expr; + } + RefPtr<Expr> ConstructLookupResultExpr( LookupResultItem const& item, RefPtr<Expr> baseExpr, @@ -228,6 +238,30 @@ namespace Slang } break; + case LookupResultItem::Breadcrumb::Kind::This: + { + // We expect a `this` to always come + // at the start of a chain. + SLANG_ASSERT(bb == nullptr); + + // The member was looked up via a `this` expression, + // so we need to create one here. + if (auto extensionDeclRef = breadcrumb->declRef.As<ExtensionDecl>()) + { + bb = createImplicitThisMemberExpr( + GetTargetType(extensionDeclRef), + loc); + } + else + { + auto type = DeclRefType::Create(getSession(), breadcrumb->declRef); + bb = createImplicitThisMemberExpr( + type, + loc); + } + } + break; + default: SLANG_UNREACHABLE("all cases handle"); } |
