Thanks for the review.  I have updated the comments, and the test<br>cases for ARM.  The new patch file is attached.<br><br>Thanks.<br>Logan<br><br><div class="gmail_quote">On Fri, Oct 19, 2012 at 7:47 AM, Eli Friedman <span dir="ltr"><<a href="mailto:eli.friedman@gmail.com" target="_blank">eli.friedman@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thu, Oct 18, 2012 at 8:23 AM, Logan Chien <<a href="mailto:tzuhsiang.chien@gmail.com">tzuhsiang.chien@gmail.com</a>> wrote:<br>

><br>
><br>
> ---------- Forwarded message ----------<br>
> From: Logan Chien <<a href="mailto:tzuhsiang.chien@gmail.com">tzuhsiang.chien@gmail.com</a>><br>
> Date: Thu, Oct 18, 2012 at 11:20 PM<br>
> Subject: Re: [cfe-commits] Bug 11709 Fix: va_list on ARM is not following<br>
> AAPCS 7.1.4<br>
> To: Eli Friedman <<a href="mailto:eli.friedman@gmail.com">eli.friedman@gmail.com</a>><br>
><br>
><br>
> Thanks for your review.<br>
><br>
> I have added new test to test/SemaCXX/builtins-arm.cpp to<br>
> specify the error message when PerformCopyInitialization fails.<br>
><br>
> BTW, is the InitializedEntity related code correct?  I hope I can<br>
> get a modifiable Lvalue reference after initialization (not a copy<br>
> of the value in the structure.)  I'm not very confident with this.<br>
<br>
</div>The code is doing the right thing.<br>
<br>
+    // If va_list is a record type and we are compiling under C++ mode,<br>
+    // then we should check the argument by copy assignment operator<br>
<br>
This comment is wrong; even though you're using<br>
PerformCopyInitialization, there's no actual copy because the<br>
InitializedEntity has reference type.  "If va_list is a record type<br>
and we are compiling in C++ mode, check the argument using reference<br>
binding."<br>
<br>
+// RUN: %clang_cc1 -triple armv7 -fsyntax-only -verify %s<br>
+<br>
+#if defined(__ARM_PCS) || defined(__ARM_EABI__)<br>
<br>
Please don't use an ifdef here; just use an appropriate triple.<br>
<br>
Otherwise, looks fine.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Eli<br>
</font></span></blockquote></div><br>