[PATCH] D132286: [clang][Interp] Implement function calls

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

tbaeder added inline comments.


================
Comment at: clang/test/AST/Interp/functions.cpp:38
+static_assert(add_second(10, 3, true) == 13, "");
+static_assert(add_second(300, -20, false) == 300, "");
----------------
shafik wrote:
> I would expect a lot more test cases: trailing return types, default arguments, overloads etc...
Things like trailing return types shouldn't make a difference here, provided they result in the same `FunctionDecl`. I've added support for `CXXDefaultArgExpr`s.


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

https://reviews.llvm.org/D132286