[cfe-commits] Bug 11709 Fix: va_list on ARM is not following AAPCS 7.1.4
jordan_rose at apple.com
Thu Oct 4 18:19:19 PDT 2012
A couple comments (sorry for the long long delay):
- AST can't depend on Sema. Is there a reason for the include of Sema.h in ASTContext.cpp?
- It would be a good idea to leave a comment in ExprClassification.cpp about why __va_list needs an exception.
- For ARMTargetInfo::getBuiltinVaListKind, maybe you can just decide this at construction time and stick it in a local variable? 'unsigned usesAAPCS : 1'?
Thanks for working on this, Weiming, Logan.
On Oct 2, 2012, at 22:34 , Logan Chien <tzuhsiang.chien at gmail.com> wrote:
> However, I think we can enhance the test cases a little.
> 1. Keep the void* test in builtins-arm.c for apcs-gnu ABI.
> 2. Add the unit test for va_list name mangling.
> ps. The attached patch is the revised patch with the updated
> test cases.
> On Thu, Sep 20, 2012 at 6:13 AM, Weiming Zhao <weimingz at codeaurora.org> wrote:
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by
> The Linux Foundation
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits