[PATCH] D139713: [Sema] Fix crash when evaluating nested call with value-dependent arg
Akira Hatanaka via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 16 18:03:50 PST 2022
ahatanak added a comment.
I read the patches and review comments in https://reviews.llvm.org/D42776, but I don't remember why I added that assert. Maybe I was just trying to ensure the version number passed to `getTemporary` was the one that was used to create the temporary. For example, if a temporary was created using version X and later retrieved using version Y because of some bug in constant evaluation, the assertion would fail.
But the assertion is wrong, so maybe it should just be removed.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139713/new/
https://reviews.llvm.org/D139713
More information about the cfe-commits
mailing list