Re: [cfe-commits] Fwd: Bug 11709 Fix: va_list on ARM is not following AAPCS 7.1.4

2012-10-20 Thread Logan Chien
Thanks. Committed as 166369. Logan On Sat, Oct 20, 2012 at 8:41 AM, Eli Friedman eli.fried...@gmail.comwrote: On Thu, Oct 18, 2012 at 9:53 PM, Logan Chien tzuhsiang.ch...@gmail.com wrote: Thanks for the review. I have updated the comments, and the test cases for ARM. The new patch

Re: [cfe-commits] Fwd: Bug 11709 Fix: va_list on ARM is not following AAPCS 7.1.4

2012-10-19 Thread David Tweed
Chien Sent: 19 October 2012 05:54 To: Eli Friedman; Weiming Zhao; cfe-commits@cs.uiuc.edu Subject: Re: [cfe-commits] Fwd: Bug 11709 Fix: va_list on ARM is not following AAPCS 7.1.4 Thanks for the review. I have updated the comments, and the test cases for ARM. The new patch file is attached. Thanks

Re: [cfe-commits] Fwd: Bug 11709 Fix: va_list on ARM is not following AAPCS 7.1.4

2012-10-19 Thread Eli Friedman
On Thu, Oct 18, 2012 at 9:53 PM, Logan Chien tzuhsiang.ch...@gmail.com wrote: Thanks for the review. I have updated the comments, and the test cases for ARM. The new patch file is attached. Looks fine; please commit. -Eli ___ cfe-commits mailing

[cfe-commits] Fwd: Bug 11709 Fix: va_list on ARM is not following AAPCS 7.1.4

2012-10-18 Thread Logan Chien
-- Forwarded message -- From: Logan Chien tzuhsiang.ch...@gmail.com Date: Thu, Oct 18, 2012 at 11:20 PM Subject: Re: [cfe-commits] Bug 11709 Fix: va_list on ARM is not following AAPCS 7.1.4 To: Eli Friedman eli.fried...@gmail.com Thanks for your review. I have added new test to

Re: [cfe-commits] Fwd: Bug 11709 Fix: va_list on ARM is not following AAPCS 7.1.4

2012-10-18 Thread Eli Friedman
On Thu, Oct 18, 2012 at 8:23 AM, Logan Chien tzuhsiang.ch...@gmail.com wrote: -- Forwarded message -- From: Logan Chien tzuhsiang.ch...@gmail.com Date: Thu, Oct 18, 2012 at 11:20 PM Subject: Re: [cfe-commits] Bug 11709 Fix: va_list on ARM is not following AAPCS 7.1.4 To:

Re: [cfe-commits] Fwd: Bug 11709 Fix: va_list on ARM is not following AAPCS 7.1.4

2012-10-18 Thread Logan Chien
Thanks for the review. I have updated the comments, and the test cases for ARM. The new patch file is attached. Thanks. Logan On Fri, Oct 19, 2012 at 7:47 AM, Eli Friedman eli.fried...@gmail.comwrote: On Thu, Oct 18, 2012 at 8:23 AM, Logan Chien tzuhsiang.ch...@gmail.com wrote: