[clang][Sema] Null-check type in resolveMemberExpr() before checking for auto type

[HighCommander4]

Fixes clangd/clangd#2301

[llvmbot]

@llvm/pr-subscribers-clang

Author: Nathan Ridge (HighCommander4)

Changes

Fixes clangd/clangd#2301

---

Full diff: https://github.com/llvm/llvm-project/pull/124628.diff

1 Files Affected:

• (modified) clang/lib/Sema/HeuristicResolver.cpp (+2)

diff --git a/clang/lib/Sema/HeuristicResolver.cpp b/clang/lib/Sema/HeuristicResolver.cpp
index 87c7274e7aefa6..8c8c5d90f8245c 100644
--- a/clang/lib/Sema/HeuristicResolver.cpp
+++ b/clang/lib/Sema/HeuristicResolver.cpp
@@ -242,6 +242,8 @@ std::vector<const NamedDecl *> HeuristicResolverImpl::resolveMemberExpr(
       BaseType = resolveExprToType(Base);
     }
   }
+  if (BaseType.isNull())
+    return {};
   if (const auto *AT = BaseType->getContainedAutoType()) {
     // If BaseType contains a dependent `auto` type, deduction will not have
     // been performed on it yet. In simple cases (e.g. `auto` variable with

[HighCommander4]

No test case because there wasn't one in the bug report; I diagnosed the issue based on the stack trace only.

[zyn0217]

LG, feel free to merge after addressing @cor3ntin 's concern