<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, Sep 14, 2016 at 10:18 AM Kuba Brecka <<a href="mailto:kuba.brecka@gmail.com">kuba.brecka@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class="gmail_msg"><div class="gmail_msg">Hi Manuel,</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">I believe you're maintaining Phabricator at <a href="http://reviews.llvm.org" class="gmail_msg" target="_blank">reviews.llvm.org</a>.  Duncan likes to send his patch comments via email, like in the example below.  Do you know why don't his replies get imported into the web interface?  The reply was sent to "<a href="mailto:reviews+D24569+public+a5763c0a090df06f@reviews.llvm.org" class="gmail_msg" target="_blank">reviews+D24569+public+a5763c0a090df06f@reviews.llvm.org</a>", and I bet that the purpose of this weird address is to match the revision this reply belongs to.  I never saw any email reply actually appear in the web interface, and that sounds like a server misconfiguration.</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Or is that something that Phabricator just doesn't support?</div></div></blockquote><div><br></div><div>Correct, phab doesn't support in-line replies. Would be great to add it, but it's quite a bit of development work.</div><div>Generally, email is the system of record, and phab exists to make email based reviews more convenient.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class="gmail_msg"><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Thanks,</div><div class="gmail_msg">Kuba</div><div class="gmail_msg"><br class="gmail_msg"><blockquote type="cite" class="gmail_msg"><div class="gmail_msg">Begin forwarded message:</div><br class="m_2809898708155487169Apple-interchange-newline gmail_msg"><div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px" class="gmail_msg"><span style="font-family:-webkit-system-font,Helvetica Neue,Helvetica,sans-serif;color:rgba(0,0,0,1.0)" class="gmail_msg"><b class="gmail_msg">From: </b></span><span style="font-family:-webkit-system-font,Helvetica Neue,Helvetica,sans-serif" class="gmail_msg">"Duncan P. N. Exon Smith" <<a href="mailto:dexonsmith@apple.com" class="gmail_msg" target="_blank">dexonsmith@apple.com</a>><br class="gmail_msg"></span></div><div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px" class="gmail_msg"><span style="font-family:-webkit-system-font,Helvetica Neue,Helvetica,sans-serif;color:rgba(0,0,0,1.0)" class="gmail_msg"><b class="gmail_msg">Subject: </b></span><span style="font-family:-webkit-system-font,Helvetica Neue,Helvetica,sans-serif" class="gmail_msg"><b class="gmail_msg">Re: [PATCH] D24569: [libcxx] Allow sanitizing libcxx with ASan+UBSan simultaneously</b><br class="gmail_msg"></span></div><div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px" class="gmail_msg"><span style="font-family:-webkit-system-font,Helvetica Neue,Helvetica,sans-serif;color:rgba(0,0,0,1.0)" class="gmail_msg"><b class="gmail_msg">Date: </b></span><span style="font-family:-webkit-system-font,Helvetica Neue,Helvetica,sans-serif" class="gmail_msg">14 September 2016 at 19:01:29 GMT+2<br class="gmail_msg"></span></div><div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px" class="gmail_msg"><span style="font-family:-webkit-system-font,Helvetica Neue,Helvetica,sans-serif;color:rgba(0,0,0,1.0)" class="gmail_msg"><b class="gmail_msg">To: </b></span><span style="font-family:-webkit-system-font,Helvetica Neue,Helvetica,sans-serif" class="gmail_msg"><a href="mailto:reviews+D24569+public+a5763c0a090df06f@reviews.llvm.org" class="gmail_msg" target="_blank">reviews+D24569+public+a5763c0a090df06f@reviews.llvm.org</a><br class="gmail_msg"></span></div><div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px" class="gmail_msg"><span style="font-family:-webkit-system-font,Helvetica Neue,Helvetica,sans-serif;color:rgba(0,0,0,1.0)" class="gmail_msg"><b class="gmail_msg">Cc: </b></span><span style="font-family:-webkit-system-font,Helvetica Neue,Helvetica,sans-serif" class="gmail_msg"><a href="mailto:kuba.brecka@gmail.com" class="gmail_msg" target="_blank">kuba.brecka@gmail.com</a>, <a href="mailto:eric@efcs.ca" class="gmail_msg" target="_blank">eric@efcs.ca</a>, <a href="mailto:compnerd@compnerd.org" class="gmail_msg" target="_blank">compnerd@compnerd.org</a>, <a href="mailto:filcab+llvm.phabricator@gmail.com" class="gmail_msg" target="_blank">filcab+llvm.phabricator@gmail.com</a>, <a href="mailto:beanz@apple.com" class="gmail_msg" target="_blank">beanz@apple.com</a>, <a href="mailto:zaks.anna@gmail.com" class="gmail_msg" target="_blank">zaks.anna@gmail.com</a>, <a href="mailto:mgorny@gentoo.org" class="gmail_msg" target="_blank">mgorny@gentoo.org</a><br class="gmail_msg"></span></div><br class="gmail_msg"><div class="gmail_msg"><div class="gmail_msg"><br class="gmail_msg"><blockquote type="cite" class="gmail_msg">On 2016-Sep-14, at 09:39, Kuba Brecka <<a href="mailto:kuba.brecka@gmail.com" class="gmail_msg" target="_blank">kuba.brecka@gmail.com</a>> wrote:<br class="gmail_msg"><br class="gmail_msg">kubabrecka added inline comments.<br class="gmail_msg"><br class="gmail_msg">================<br class="gmail_msg">Comment at: test/libcxx/test/config.py:628<br class="gmail_msg">@@ +627,3 @@<br class="gmail_msg">+                    self.cxx.flags += ['-fsanitize=undefined',<br class="gmail_msg"></blockquote><br class="gmail_msg">Shouldn't this be '-fsanitize="undefined;address"'?<br class="gmail_msg"><br class="gmail_msg"><blockquote type="cite" class="gmail_msg">+                                       '-fno-sanitize=vptr,function,float-divide-by-zero',<br class="gmail_msg">+                                       '-fno-sanitize-recover=all']<br class="gmail_msg"></blockquote><br class="gmail_msg">Can this be refactored to avoid code duplication with the UBSan changes?<br class="gmail_msg"></div></div></blockquote></div><br class="gmail_msg"></div></blockquote></div></div>