tree a6b25c1715afc8512a37e3f0f10cbdf4be07d5b3
parent 6304c0836a4dd2d77373d45716092b2a088fa948
author Alex Hoppen <ahoppen@apple.com> 1620331886 -0700
committer Argyrios Kyrtzidis <kyrtzidis@apple.com> 1620331946 -0700

[Index] Ignore nullptr decls for indexing

We can end up with a call to `indexTopLevelDecl(D)` with `D == nullptr` in non-assert builds e.g. when indexing a module in `indexModule` and
- `ASTReader::GetDecl` returns `nullptr` if `Index >= DeclsLoaded.size()`, thus returning `nullptr`
=> `ModuleDeclIterator::operator*` returns `nullptr`
=> we call `IndexCtx.indexTopLevelDecl` with `nullptr`

Be resilient and just ignore the `nullptr` decls during indexing.

Reviewed By: akyrtzi

Differential Revision: https://reviews.llvm.org/D102001