[PATCH] D65573: Add User docs for ASTImporter
Gabor Marton via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 16 05:22:37 PDT 2019
martong marked 3 inline comments as done.
martong added a comment.
In D65573#1625881 <https://reviews.llvm.org/D65573#1625881>, @a_sidorin wrote:
> That's incredible. Thank you!
Thanks Alexei for the review! I commited a fix for the typos.
================
Comment at: cfe/trunk/docs/LibASTImporter.rst:215
+ Node *Result =
+ const_cast<Node *>(MatchRes[0].template getNodeAs<Node>("bindStr"));
+ assert(Result);
----------------
a_sidorin wrote:
> We can avoid const_cast if we change the example function signature to `const Node *`.
Indeed... Still, I did not change that to keep consistency with other parts of the examples.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65573/new/
https://reviews.llvm.org/D65573
More information about the cfe-commits
mailing list