Patch for Bug 22877

Arthur Eubanks via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 10 22:32:15 PST 2017


Test updated.


> On 9 Jan 2017, at 3:48 PM, Akira Hatanaka <ahatanaka at apple.com> 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: call void @_ZN3barC1E3foo
> // CHECK-NOT: br
> // CHECK: call void @_ZN3fooD1Ev
> // CHECK: br [[LOOP_BODY]]
> 
>> On Jan 8, 2017, at 11:34 PM, Arthur Eubanks via cfe-commits <cfe-commits at lists.llvm.org <mailto:cfe-commits at lists.llvm.org>> wrote:
>> 
>> 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 <https://llvm.org/bugs/show_bug.cgi?id=22877> regarding cleaning up default arguments in constructors while initializing an array.
>> It doesn't break any old tests.
>> 
>> Thanks,
>> Arthur Eubanks
>> 
>> <patch.diff>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at lists.llvm.org <mailto:cfe-commits at lists.llvm.org>
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170110/5eb3964a/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch.diff
Type: application/octet-stream
Size: 2095 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170110/5eb3964a/attachment.obj>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170110/5eb3964a/attachment-0001.html>


More information about the cfe-commits mailing list