[PATCH] D141497: [clang][Interp] Record initialization via conditional operator

[Timm Bäder via Phabricator via cfe-commits]

tbaeder added inline comments.


================
Comment at: clang/lib/AST/Interp/ByteCodeExprGen.h:200
+  using ExprVisitorFunc = std::function<bool(const Expr *)>;
+  bool visitConditional(const AbstractConditionalOperator *E,
+                        ExprVisitorFunc VisitFunc);
----------------
tbaeder wrote:
> erichkeane wrote:
> > I'd probably rather make this a template taking a functor, rather than bringing in the atrocities that come with std::function.
> Alright, will look into that
Done. Not sure if this is exactly how you meant it to look. Feels a bit inelegant.


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

https://reviews.llvm.org/D141497