lebedev.ri added a comment. Sounds good, thank you!
================ Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize-loop-convert.rst:262 +not been used on code with a compatibility requirements of OpenMP prior to +version 5. ---------------- I would add a cross-reference to `NOLINT` documentation, and a mention that the check **intentionally** makes no attempt to prevent issuing these diagnostics for OpenMP loops. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61827/new/ https://reviews.llvm.org/D61827 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
