Author: Rahul Joshi Date: 2024-09-13T07:51:17-07:00 New Revision: f637273d7708da0a01c3adb2bb85ae1c541f46a1
URL: https://github.com/llvm/llvm-project/commit/f637273d7708da0a01c3adb2bb85ae1c541f46a1 DIFF: https://github.com/llvm/llvm-project/commit/f637273d7708da0a01c3adb2bb85ae1c541f46a1.diff LOG: [clang][TableGen] Change SACheckersEmitter to use const RecordKeeper (#108477) Change SACheckersEmitter to use const RecordKeeper. This is a part of effort to have better const correctness in TableGen backends: https://discourse.llvm.org/t/psa-planned-changes-to-tablegen-getallderiveddefinitions-api-potential-downstream-breakages/81089 Added: Modified: clang/utils/TableGen/ClangSACheckersEmitter.cpp clang/utils/TableGen/TableGenBackends.h Removed: ################################################################################ diff --git a/clang/utils/TableGen/ClangSACheckersEmitter.cpp b/clang/utils/TableGen/ClangSACheckersEmitter.cpp index 2a2e466ae19797..44c2d8b31655d2 100644 --- a/clang/utils/TableGen/ClangSACheckersEmitter.cpp +++ b/clang/utils/TableGen/ClangSACheckersEmitter.cpp @@ -174,9 +174,11 @@ static void printOption(llvm::raw_ostream &OS, StringRef FullName, OS << "true"; } -void clang::EmitClangSACheckers(RecordKeeper &Records, raw_ostream &OS) { - std::vector<Record*> checkers = Records.getAllDerivedDefinitions("Checker"); - std::vector<Record*> packages = Records.getAllDerivedDefinitions("Package"); +void clang::EmitClangSACheckers(const RecordKeeper &Records, raw_ostream &OS) { + ArrayRef<const Record *> checkers = + Records.getAllDerivedDefinitions("Checker"); + ArrayRef<const Record *> packages = + Records.getAllDerivedDefinitions("Package"); using SortedRecords = llvm::StringMap<const Record *>; diff --git a/clang/utils/TableGen/TableGenBackends.h b/clang/utils/TableGen/TableGenBackends.h index 79e9be6c031927..4e59653fac06da 100644 --- a/clang/utils/TableGen/TableGenBackends.h +++ b/clang/utils/TableGen/TableGenBackends.h @@ -94,7 +94,8 @@ void EmitClangDiagGroups(const llvm::RecordKeeper &Records, void EmitClangDiagsIndexName(const llvm::RecordKeeper &Records, llvm::raw_ostream &OS); -void EmitClangSACheckers(llvm::RecordKeeper &Records, llvm::raw_ostream &OS); +void EmitClangSACheckers(const llvm::RecordKeeper &Records, + llvm::raw_ostream &OS); void EmitClangCommentHTMLTags(const llvm::RecordKeeper &Records, llvm::raw_ostream &OS); _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
