[PATCH] D75446: [clang][Syntax] Handle macro arguments in spelledForExpanded

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 16 07:35:33 PDT 2020


sammccall added a comment.

I think per offline discussion, this was going to have some additional docs clarifying the contracts of this function, and maybe extra tests?

The conclusion IIRC was that this function is designed to find the text that you could replace in order to replace an AST node, so it's fairly conservative (but arg expansions are fair game assuming expanded once).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75446





More information about the cfe-commits mailing list