[PATCH] D151553: [clang] Fix consteval operators in template contexts

Mariya Podchishchaeva via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 26 05:06:59 PDT 2023


Fznamznon added inline comments.


================
Comment at: clang/lib/Sema/TreeTransform.h:11940-11943
-  ExprResult Callee = getDerived().TransformExpr(E->getCallee());
-  if (Callee.isInvalid())
-    return ExprError();
-
----------------
cor3ntin wrote:
> I don't understand why we would not need to transform the callee. do we do that elsewhere?
For example, the code above for call and subscript operators never transforms the callee.
This `TransFormExpr` call  seems to be a no-op for overloaded operator call case, and the result never ends up in the resulting ast.



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151553/new/

https://reviews.llvm.org/D151553



More information about the cfe-commits mailing list