[PATCH] D59094: [ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 15 12:21:55 PDT 2019


efriedma accepted this revision.
efriedma added a reviewer: t.p.northover.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: clang/lib/CodeGen/TargetInfo.cpp:6023
+  // Variadic functions should always marshal to the base standard.
+  bool isAAPCS_VFP =
+      !isVariadic && isEffectivelyAAPCS_VFP(functionCallConv, /* AAPCS16 */ true);
----------------
Variable name should be capitalized.


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

https://reviews.llvm.org/D59094





More information about the cfe-commits mailing list