shafik added inline comments.
================
Comment at: clang/lib/AST/Interp/Interp.h:1315
+ if (InterpretBuiltin(S, PC, Func->getBuiltinID())) {
+ NewFrame.release();
+ return true;
----------------
We don't have to update `S.Current`?
================
Comment at: clang/lib/AST/Interp/InterpBuiltin.cpp:20
+template <PrimType Name, class T = typename PrimConv<Name>::T>
+static bool Ret(InterpState &S, CodePtr &PC) {
+ S.CallStackDepth--;
----------------
Why not just factor out `Ret` now?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137487/new/
https://reviews.llvm.org/D137487
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
