<div dir="ltr">Hi, Sam,<div>many thanks for the patch, </div><div>I'm sorry I didn't notice it earlier.</div><div>Do you mind uploading your patch to Phabricator, see <a href="https://llvm.org/docs/Phabricator.html">https://llvm.org/docs/Phabricator.html</a> for instructions, </div><div>so it would be easier to review / discuss the changes. </div><div>Kind regards,</div><div>Alexander Shaposhnikov</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jul 9, 2017 at 2:11 PM, Sam Conrad via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div>This adds a warning emitted by clang-reorder-fields when the reordering fields breaks dependencies in the initializer list (such that -Wuninitialized would warn).  For example, given:<br></div>Foo::Foo(int x)<br></div>  : a(x)<br></div>  , b(a) {}<br><br></div><div>Reordering fields to [b,a] gives:<br>Foo::Foo(int x)<br></div><div>  : b(a)<br></div><div>  , a(x) {}</div><div><br></div>Emits the warning:<br>2: Warning: reordering field a after b makes a uninitialized when used in init expression.<br><br></div>The patch also reformats a few lines that were over 80 columns wide.<br><div><br></div></div>
<br>______________________________<wbr>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>