This revision was automatically updated to reflect the committed changes. Closed by commit rG775ca3a89cba: [clang-tidy] Fix a crash for raw-string-literal check. (authored by hokein).
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D102770/new/ https://reviews.llvm.org/D102770 Files: clang-tools-extra/clang-tidy/modernize/RawStringLiteralCheck.cpp Index: clang-tools-extra/clang-tidy/modernize/RawStringLiteralCheck.cpp =================================================================== --- clang-tools-extra/clang-tidy/modernize/RawStringLiteralCheck.cpp +++ clang-tools-extra/clang-tidy/modernize/RawStringLiteralCheck.cpp @@ -56,7 +56,7 @@ *Result.SourceManager, Result.Context->getLangOpts()); StringRef Text = Lexer::getSourceText(CharRange, *Result.SourceManager, Result.Context->getLangOpts()); - if (isRawStringLiteral(Text)) + if (Text.empty() || isRawStringLiteral(Text)) return false; return containsEscapes(Text, R"('\"?x01)");
Index: clang-tools-extra/clang-tidy/modernize/RawStringLiteralCheck.cpp =================================================================== --- clang-tools-extra/clang-tidy/modernize/RawStringLiteralCheck.cpp +++ clang-tools-extra/clang-tidy/modernize/RawStringLiteralCheck.cpp @@ -56,7 +56,7 @@ *Result.SourceManager, Result.Context->getLangOpts()); StringRef Text = Lexer::getSourceText(CharRange, *Result.SourceManager, Result.Context->getLangOpts()); - if (isRawStringLiteral(Text)) + if (Text.empty() || isRawStringLiteral(Text)) return false; return containsEscapes(Text, R"('\"?x01)");
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits