[PATCH] D84348: WIP: Add complete id-expression support to syntax trees

Eduardo Caldas via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 27 07:09:36 PDT 2020


eduucaldas marked 2 inline comments as done.
eduucaldas added inline comments.


================
Comment at: clang/lib/Tooling/Syntax/BuildTree.cpp:834
+  // FIXME: I feel like this could be upstreamed.
+  SourceRange getUnqualifiedIdSourceRange(DeclRefExpr *S) {
+    if (S->hasExplicitTemplateArgs())
----------------
gribozavr2 wrote:
> WDYM by "upstream"?
I meant to put that logic under the DeclRefExpr node, instead of here. But I found a way of writing this logic in a simpler way :). So I just inlined it!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84348





More information about the cfe-commits mailing list