ymandel marked 5 inline comments as done. ymandel added a comment. Thanks for the review!
================ Comment at: clang/include/clang/Tooling/Transformer/Stencil.h:96 +// Constructs an expression that idiomatically represents a value, taking into +// account whether `ExprId` is a pointer or already a value. +Stencil asValue(llvm::StringRef ExprId); ---------------- gribozavr2 wrote: > Need more explanation to cover the cases when the expression is a pointer to > a pointer (do we deref once or twice?), or a smart pointer (do we take care > of that at all?) > > WDYT about calling it `maybeDeref()` or `derefIfPointer()`? I went with maybe... Given that, I didn't elaborate on pointer-to-pointer case because it never claims to result in a value. WDYT? also, I kind of like derefIfPointer better, but addressOfIfValue just seemed to clunky. So, went w/ maybe... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70554/new/ https://reviews.llvm.org/D70554 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits