tbaeder added inline comments.
================ Comment at: clang/docs/ReleaseNotes.rst:290 + number of code lines it prints has been increased from 1 to 16. This + can be controlled using ``-fcaret-diagnostics-max-lines=``. ---------------- dyung wrote: > Is this correct? I notice that all of the tests you update used the form of > the option "-fcaret-diagnostics-max-lines n" rather than > "-fcaret-diagnostics-max-lines=n". When I tried to update some private tests > to use "-fcaret-diagnostics-max-lines=n", the compiler did not accept it, but > when I changed it to use "-fcaret-diagnostics-max-lines n" it worked. I think > we either need to update the option to accept both forms or to correct the > documentation. Yes, you're right, that should be changed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147875/new/ https://reviews.llvm.org/D147875 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
