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

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 2 11:11:11 PDT 2022


rjmccall added inline comments.


================
Comment at: clang/lib/CodeGen/TargetInfo.cpp:2457
+                    /*IsVectorCall=*/false, /*IsRegCall=*/false);
+  }
+
----------------
Hmm.  Doesn't EC ABI lowering need to preserve this same state, or else you'll get incompatibilities when you exhaust SSE registers?

Should you just be calling over to this at a higher-level point, like in `computeInfo`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125419



More information about the cfe-commits mailing list