ABataev added inline comments.
================ Comment at: clang/include/clang/AST/OpenMPClause.h:8893-8894 + + /// Sets the location of '('. + void setLParenLoc(SourceLocation Loc) { LParenLoc = Loc; } + ---------------- abidmalikwaterloo wrote: > ABataev wrote: > > make it private > Any reason behind this? I went through other classes, this function is public > in some and private in some. The AST is immutable, such functions are/must be used in very rare places. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122255/new/ https://reviews.llvm.org/D122255 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits