[clang] [clang] SourceLocIdentKind::Function should not be dependent (PR #94942)

[Younan Zhang via cfe-commits]

zyn0217 wrote:

nit: Please rephrase the commit message before you merge this patch. A large piece of AST involving memory addresses is really hard to decipher.

In addition, it would be nice if you can simplify the test case further - it's super unclear why a bunch of templates are there outside of the `SourceLocation::Current()` - probably one or two is sufficient?

https://github.com/llvm/llvm-project/pull/94942