[PATCH] D85621: [clang] Allow DynTypedNode to store a TemplateArgumentLoc

Nathan Ridge via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 10 21:42:20 PDT 2020


nridge added a comment.

In D85621#2205944 <https://reviews.llvm.org/D85621#2205944>, @njames93 wrote:

> In D85621#2205856 <https://reviews.llvm.org/D85621#2205856>, @nridge wrote:
>
>> In D85621#2205803 <https://reviews.llvm.org/D85621#2205803>, @hokein wrote:
>>
>>> Please run `clang/docs/tools/dump_ast_matchers.py` script to update the `LibASTMatchersReference.html` file.
>>
>> Done. Note, it looks like this script needs to be run with python 2. Should we document this somewhere?
>
> Given that python2 is EOL I think the correct course of action is to make it work with python3.

Filed https://bugs.llvm.org/show_bug.cgi?id=47102 for this.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85621/new/

https://reviews.llvm.org/D85621



More information about the cfe-commits mailing list