Re: r302750 - PR22877: When constructing an array via a constructor with a default argument

2017-05-12 Thread Galina Kistanova via cfe-commits
I will look at this. Thanks Galina On Fri, May 12, 2017 at 2:00 PM, Richard Smith wrote: > On 12 May 2017 at 13:24, Renato Golin wrote: > >> On 12 May 2017 at 20:46, Diana Picus wrote: >> > On 11 May 2017 at 21:14, Richard Smith wrote: >> >> Thanks for the revert, fixed up and recommitted i

Re: r302750 - PR22877: When constructing an array via a constructor with a default argument

2017-05-12 Thread Richard Smith via cfe-commits
On 12 May 2017 at 13:24, Renato Golin wrote: > On 12 May 2017 at 20:46, Diana Picus wrote: > > On 11 May 2017 at 21:14, Richard Smith wrote: > >> Thanks for the revert, fixed up and recommitted in r302817. > >> > >> Any idea why not a single buildbot sent me any email about this? :( (A > >> cou

Re: r302750 - PR22877: When constructing an array via a constructor with a default argument

2017-05-12 Thread Renato Golin via cfe-commits
On 12 May 2017 at 20:46, Diana Picus wrote: > On 11 May 2017 at 21:14, Richard Smith wrote: >> Thanks for the revert, fixed up and recommitted in r302817. >> >> Any idea why not a single buildbot sent me any email about this? :( (A >> couple of them were red at the previous change too, but some o

Re: r302750 - PR22877: When constructing an array via a constructor with a default argument

2017-05-12 Thread Diana Picus via cfe-commits
On 11 May 2017 at 21:14, Richard Smith wrote: > Thanks for the revert, fixed up and recommitted in r302817. > > Any idea why not a single buildbot sent me any email about this? :( (A > couple of them were red at the previous change too, but some of them were > not (eg. clang-cmake-armv7-a15). Hm,

Re: r302750 - PR22877: When constructing an array via a constructor with a default argument

2017-05-11 Thread Richard Smith via cfe-commits
Thanks for the revert, fixed up and recommitted in r302817. Any idea why not a single buildbot sent me any email about this? :( (A couple of them were red at the previous change too, but some of them were not (eg. clang-cmake-armv7-a15). On 11 May 2017 at 01:28, Diana Picus wrote: > Hexagon too

Re: r302750 - PR22877: When constructing an array via a constructor with a default argument

2017-05-11 Thread Diana Picus via cfe-commits
Hexagon too: http://lab.llvm.org:8011/builders/clang-hexagon-elf/builds/7916 On 11 May 2017 at 10:27, Diana Picus wrote: > Hi Richard, > > I reverted this and its fixup in r302776 since some of the ARM bots > were still broken: > http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15/builds/6969

Re: r302750 - PR22877: When constructing an array via a constructor with a default argument

2017-05-11 Thread Diana Picus via cfe-commits
Hi Richard, I reverted this and its fixup in r302776 since some of the ARM bots were still broken: http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15/builds/6969 http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-a15/builds/6959 http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15-full/

r302750 - PR22877: When constructing an array via a constructor with a default argument

2017-05-10 Thread Richard Smith via cfe-commits
Author: rsmith Date: Wed May 10 19:17:17 2017 New Revision: 302750 URL: http://llvm.org/viewvc/llvm-project?rev=302750&view=rev Log: PR22877: When constructing an array via a constructor with a default argument in list-initialization, run cleanups for the default argument after each iteration of t