[PATCH] D155548: [clang][ExprConst] Short-circuit ConstantExpr evaluation

[Corentin Jabot via Phabricator via cfe-commits]

cor3ntin added a comment.

I think this does make sense. Have you notice an impact on performance?
I wonder if there are places where we should cache the result of evaluating a ConstantExpr, like in `ExprEvaluatorBase::VisitConstantExpr`
How often do we construct a ConstantExpr without an initial computed value?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155548