Re: Patch for Bug 22877

2017-01-11 Thread Arthur Eubanks via cfe-commits
Test updated. patch.diff Description: Binary data On 9 Jan 2017, at 3:48 PM, Akira Hatanaka wrote:Perhaps you can check more explicitly that the destructor is called inside the loop?// CHECK: [[regex for LOOP_BODY]]:// CHECK: call void @_ZN3fooC1Ev// CHECK-NOT: br// CHECK: ca

Re: Patch for Bug 22877

2017-01-09 Thread Akira Hatanaka via cfe-commits
Perhaps you can check more explicitly that the destructor is called inside the loop? // CHECK: [[regex for LOOP_BODY]]: // CHECK: call void @_ZN3fooC1Ev // CHECK-NOT: br // CHECK: call void @_ZN3barC1E3foo // CHECK-NOT: br // CHECK: call void @_ZN3fooD1Ev // CHECK: br [[LOOP_BODY]] > On Jan 8, 2

Patch for Bug 22877

2017-01-09 Thread Arthur Eubanks via cfe-commits
Hi,This is my first commit, please feel free to critique any of the code/new test since I'm not quite sure if this is in the correct format.The patch is for the bug https://llvm.org/bugs/show_bug.cgi?id=22877 regarding cleaning up default arguments in constructors while initializing an array.It doe