[PATCH] D60029: Add const children() accessors to Stmts

Nicolas Manichon via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Mar 30 11:05:47 PDT 2019


nicolas added a comment.

In D60029#1448938 <https://reviews.llvm.org/D60029#1448938>, @riccibruno wrote:

> Did you go over all the statements on `Stmt.h` systematically ?


Yes.

> You could also do the same thing for all of the statements/expressions in `StmtCXX.h`, `Expr.h`, `ExprCXX.h`, and so on.

I will go over those soon. I believe that `Expr.h` is mostly done.

> Apart from this did you run `clang-format` on the patch ?

I tried to stick to the style of the non-const functions, even when it was wrong. I will `clang-format` the next patch!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60029





More information about the cfe-commits mailing list