riccibruno added a comment. In D60029#1448944 <https://reviews.llvm.org/D60029#1448944>, @nicolas wrote:
> 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! Ah I see. Yeah I know that a fair amount of code is not formatted unfortunately. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60029/new/ https://reviews.llvm.org/D60029 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
