[PATCH] D156244: [clang] Do not crash on use of a variadic overloaded operator
Mariya Podchishchaeva via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 25 09:23:40 PDT 2023
Fznamznon added inline comments.
================
Comment at: clang/lib/Sema/SemaOverload.cpp:14001
+
+ if (FnDecl->isInvalidDecl())
+ return ExprError();
----------------
shafik wrote:
> It feels a bit weird that we are succeeding in finding the best viable function and what we find is invalid.
Yes, but it seems to be done this way in other places as well:
https://github.com/llvm/llvm-project/blob/0478ef2d366c6f88678e37d54190dcdaa0ec69da/clang/lib/Sema/SemaOverload.cpp#L15145 .
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156244/new/
https://reviews.llvm.org/D156244
More information about the cfe-commits
mailing list