[PATCH] D125419: [Arm64EC 7/?] clang side of Arm64EC varargs ABI.

David Spickett via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 21 06:17:21 PDT 2022


DavidSpickett added a comment.

I don't have the expertise to approve, but one question.

Why is there more testing in the `.c` than in `.cpp`. Is the same logic being used for both so there's no point checking twice?

If so what things is the `.cpp` test looking for specifically? I struggle to see how the lines it checks for map into the call to the variadic function. Is it looking at the mangling, the choice of pointer or not?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125419



More information about the llvm-commits mailing list