[llvm-dev] Comments sent via mail are not imported into Phabricator web

Kuba Brecka via llvm-dev llvm-dev at lists.llvm.org
Wed Sep 14 10:18:15 PDT 2016

Hi Manuel,

I believe you're maintaining Phabricator at reviews.llvm.org <http://reviews.llvm.org/>.  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 "reviews+D24569+public+a5763c0a090df06f at reviews.llvm.org <mailto:reviews+D24569+public+a5763c0a090df06f at reviews.llvm.org>", 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.

Or is that something that Phabricator just doesn't support?


> Begin forwarded message:
> From: "Duncan P. N. Exon Smith" <dexonsmith at apple.com>
> Subject: Re: [PATCH] D24569: [libcxx] Allow sanitizing libcxx with ASan+UBSan simultaneously
> Date: 14 September 2016 at 19:01:29 GMT+2
> To: reviews+D24569+public+a5763c0a090df06f at reviews.llvm.org
> Cc: kuba.brecka at gmail.com, eric at efcs.ca, compnerd at compnerd.org, filcab+llvm.phabricator at gmail.com, beanz at apple.com, zaks.anna at gmail.com, mgorny at gentoo.org
>> On 2016-Sep-14, at 09:39, Kuba Brecka <kuba.brecka at gmail.com> wrote:
>> kubabrecka added inline comments.
>> ================
>> Comment at: test/libcxx/test/config.py:628
>> @@ +627,3 @@
>> +                    self.cxx.flags += ['-fsanitize=undefined',
> Shouldn't this be '-fsanitize="undefined;address"'?
>> +                                       '-fno-sanitize=vptr,function,float-divide-by-zero',
>> +                                       '-fno-sanitize-recover=all']
> Can this be refactored to avoid code duplication with the UBSan changes?

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20160914/4c6331cb/attachment.html>

More information about the llvm-dev mailing list