[clang] [clang] [diagnostics] Add `-fdiagnostics-add-output` switch for SARIF (PR #185201)
Dave Bartolomeo via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 9 09:32:09 PDT 2026
================
@@ -439,10 +439,30 @@ json::Object SarifDocumentWriter::createDocument() {
endRun();
json::Object Doc{
- {"$schema", SchemaURI},
- {"version", SchemaVersion},
+ {"$schema", Version.SchemaURI},
+ {"version", Version.SchemaVersion},
};
if (!Runs.empty())
Doc["runs"] = json::Array(Runs);
return Doc;
}
+
+ArrayRef<SarifVersion> SarifDocumentWriter::getSupportedVersions() {
+ static const SarifVersion Versions[] = {
+ {"2.1.0", "2.1",
+ "https://docs.oasis-open.org/sarif/sarif/v2.1.0/cos02/schemas/"
+ "sarif-schema-2.1.0.json",
+ true}};
+
+ ArrayRef<SarifVersion> VersionsArray = Versions;
+ assert(std::count_if(VersionsArray.begin(), VersionsArray.end(),
+ std::mem_fn(&SarifVersion::isDefault)) == 1);
+
+ return VersionsArray;
+}
+
+const SarifVersion &SarifDocumentWriter::getDefaultVersion() {
+ const auto Versions = getSupportedVersions();
+ return *std::find_if(Versions.begin(), Versions.end(),
----------------
dbartol wrote:
Ah, much nicer. Thanks!
https://github.com/llvm/llvm-project/pull/185201
More information about the cfe-commits
mailing list