<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""></div><br class=""><div><blockquote type="cite" class=""><div class="">On 9 Jan 2017, at 3:48 PM, Akira Hatanaka <<a href="mailto:ahatanaka@apple.com" class="">ahatanaka@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" content="text/html charset=us-ascii" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Perhaps you can check more explicitly that the destructor is called inside the loop?</div><div class=""><br class=""></div><div class="">// CHECK: [[regex for LOOP_BODY]]:</div><div class=""><div class="">// CHECK: call void @_ZN3fooC1Ev</div><div class="">// CHECK-NOT: br</div><div class="">// CHECK: call void @_ZN3barC1E3foo</div><div class="">// CHECK-NOT: br</div><div class="">// CHECK: call void @_ZN3fooD1Ev</div><div class="">// CHECK: br [[LOOP_BODY]]</div></div><div class=""><br class=""></div><div class=""><blockquote type="cite" class=""><div class="">On Jan 8, 2017, at 11:34 PM, Arthur Eubanks via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" content="text/html charset=us-ascii" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi,<div class=""><br class=""></div><div class="">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.</div><div class="">The patch is for the bug <a href="https://llvm.org/bugs/show_bug.cgi?id=22877" class="">https://llvm.org/bugs/show_bug.cgi?id=22877</a> regarding cleaning up default arguments in constructors while initializing an array.</div><div class="">It doesn't break any old tests.</div><div class=""><br class=""></div><div class="">Thanks,</div><div class="">Arthur Eubanks</div><div class=""><br class=""></div><div class=""></div></div><span id="cid:4CBAD9DE-43EA-4B4C-8BBB-D03E9E528E2D@apple.com" class=""><patch.diff></span><meta http-equiv="Content-Type" content="text/html charset=us-ascii" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""></div></div>_______________________________________________<br class="">cfe-commits mailing list<br class=""><a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a><br class=""><a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" class="">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br class=""></div></blockquote></div><br class=""></div></div></blockquote></div><br class=""></body></html>