[PATCH] D28672: [ASTMatchers] update doc by running dump_ast_matchers.py

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 15 13:23:56 PST 2017


On Sun, Jan 15, 2017 at 4:19 PM, Eric Liu <ioeric at google.com> wrote:
> I've sent an email to cfe-dev (and Tanya) about this. And this patch will be
> abandoned since it doesn't fix anything.

Thank you for taking care of this!

~Aaron

>
> On Sun, Jan 15, 2017 at 10:04 PM Aaron Ballman <aaron.ballman at gmail.com>
> wrote:
>>
>> On Sun, Jan 15, 2017 at 1:00 PM, Eric Liu via Phabricator
>> <reviews at reviews.llvm.org> wrote:
>> > ioeric added a comment.
>> >
>> > In https://reviews.llvm.org/D28672#646336, @malcolm.parsons wrote:
>> >
>> >> In https://reviews.llvm.org/D28672#646151, @aaron.ballman wrote:
>> >>
>> >> > I'm not seeing anything wrong, per se, but why has so much of this
>> >> > file changed recently?
>> >>
>> >>
>> >> The script looks for doxygen docs on clang.llvm.org. Are they missing?
>> >
>> >
>> > Manuel pointed out that these changes were caused by broken links (e.g.
>> > http://clang.llvm.org/doxygen/classclang_1_1Decl.html). Some pages have been
>> > unreachable for at least a few days now.
>>
>> If links were broken with the recent server switch, we should ask
>> Tanya about them (the move was not intended to break anything). Or are
>> you saying that these changes are intended to be what fixes those
>> broken links?
>>
>> ~Aaron


More information about the cfe-commits mailing list