[patch] If a function decl cannot be merged, mark it as invalid.

Nico Weber thakis at chromium.org
Fri Jan 16 14:00:05 PST 2015


On Fri, Jan 16, 2015 at 1:52 PM, Reid Kleckner <rnk at google.com> wrote:

> On Fri, Jan 16, 2015 at 1:13 PM, Nico Weber <thakis at chromium.org> wrote:
>
>> ps: I tried to upload this to phab since this looks like the type of
>> patch it's good for, but it didn't want to accept richard at metafoo.co.uk
>> (or richard-llvm at metafoo.co.uk) in the "Reviewers" field; there also
>> weren't any "Richard Smith"s in the autocomplete field :-/
>>
>
> Phab usually wants svn committer ids, like rnk, rsmith, thakis, compnerd,
> etc.
>

Aha, thanks! Phab does seem to know that "rsmith" is also called "Richard
Smith", at least it shows that in parens in the autocomplete menu. It'd be
cool if it could autocomplete "rsmith" if I type in "Richard Smith", given
that it has the data for that. Manuel, could you route that upstream?

If anyone would rather look at this diff on phab, it's now also on
http://reviews.llvm.org/D7021
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150116/03746f02/attachment.html>


More information about the cfe-commits mailing list