[clang] [clang][AST] Remove HasFirstArg assertion in CallExpr::getBeginLoc() (PR #130725)

[Nathan Ridge via cfe-commits]

HighCommander4 wrote:

> Missing changelog.

Added.

> Otherwise, I think it makes sense even if I sort of hate it, as it relies on `TryCopyInitialization` looking at the location and nothing else.
> 
> I wonder if we could get rid of `CallExpr::CreateTemporary` and instead create something like an `OpaqueValueExpr` for the purpose of determining an initialization sequence.
> 
> (We should only need some expression with a type, a location, and a value category, afaik)

Filed https://github.com/llvm/llvm-project/issues/130824 to track this idea.

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