xgupta added inline comments.
================ Comment at: clang/docs/LibASTMatchersTutorial.rst:34 - git clone https://github.com/martine/ninja.git - cd ninja - git checkout release ---------------- SamuelMarks wrote: > xgupta wrote: > > why removed `cd ninja` & `git checkout release`? > @xgupta To match how ninja is installed everywhere else in LLVM guides. Also > ninja's release branch is almost a year old, whereas its master is 5 days old. fine, but we need to cd into ninja directory to run configure step, right? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111490/new/ https://reviews.llvm.org/D111490 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
