[PATCH] D43089: clang: Add ARCTargetInfo

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 14 10:25:35 PST 2018


rjmccall added inline comments.


================
Comment at: clang/lib/CodeGen/TargetInfo.cpp:8185
+                                            unsigned FreeRegs) const {
+  // TODO: C++ ABI?
+  unsigned SizeInRegs = (getContext().getTypeSize(Ty) + 31) / 32;
----------------
Please do go ahead and add the right logic for the C++ ABI in both this and the return-type classification.


https://reviews.llvm.org/D43089





More information about the cfe-commits mailing list