Index: test/Analysis/diagnostics/Inputs/expected-sarif/sarif-diagnostics-taint-test.c.sarif =================================================================== --- test/Analysis/diagnostics/Inputs/expected-sarif/sarif-diagnostics-taint-test.c.sarif +++ test/Analysis/diagnostics/Inputs/expected-sarif/sarif-diagnostics-taint-test.c.sarif @@ -7,7 +7,7 @@ "fileLocation": { "uri": "file:sarif-diagnostics-taint-test.c" }, - "length": 415, + "length": 519, "mimeType": "text/plain", "roles": [ "resultFile" Index: test/Analysis/diagnostics/Inputs/expected-sarif/sarif-multi-diagnostic-test.c.sarif =================================================================== --- test/Analysis/diagnostics/Inputs/expected-sarif/sarif-multi-diagnostic-test.c.sarif +++ test/Analysis/diagnostics/Inputs/expected-sarif/sarif-multi-diagnostic-test.c.sarif @@ -7,7 +7,7 @@ "fileLocation": { "uri": "file:sarif-multi-diagnostic-test.c" }, - "length": 667, + "length": 771, "mimeType": "text/plain", "roles": [ "resultFile" Index: test/Analysis/diagnostics/sarif-diagnostics-taint-test.c =================================================================== --- test/Analysis/diagnostics/sarif-diagnostics-taint-test.c +++ test/Analysis/diagnostics/sarif-diagnostics-taint-test.c @@ -1,4 +1,4 @@ -// RUN: %clang_analyze_cc1 -analyzer-checker=alpha.security.taint,debug.TaintTest %s -verify -analyzer-output=sarif -o - | %diff_sarif %S/Inputs/expected-sarif/sarif-diagnostics-taint-test.c.sarif - +// RUN: %clang_analyze_cc1 -analyzer-checker=alpha.security.taint,debug.TaintTest %s -verify -analyzer-output=sarif -o %t && echo >>%t && %normalize_sarif <%S/Inputs/expected-sarif/sarif-diagnostics-taint-test.c.sarif >%t.expected.sed.sarif && %normalize_sarif <%t | diff -U1 -b %t.expected.sed.sarif - #include "../Inputs/system-header-simulator.h" int atoi(const char *nptr); Index: test/Analysis/diagnostics/sarif-multi-diagnostic-test.c =================================================================== --- test/Analysis/diagnostics/sarif-multi-diagnostic-test.c +++ test/Analysis/diagnostics/sarif-multi-diagnostic-test.c @@ -1,4 +1,4 @@ -// RUN: %clang_analyze_cc1 -analyzer-checker=core,alpha.security.taint,debug.TaintTest %s -verify -analyzer-output=sarif -o - | %diff_sarif %S/Inputs/expected-sarif/sarif-multi-diagnostic-test.c.sarif - +// RUN: %clang_analyze_cc1 -analyzer-checker=core,alpha.security.taint,debug.TaintTest %s -verify -analyzer-output=sarif -o %t && echo >>%t && %normalize_sarif <%S/Inputs/expected-sarif/sarif-multi-diagnostic-test.c.sarif >%t.expected.sed.sarif && %normalize_sarif <%t | diff -U1 -b %t.expected.sed.sarif - #include "../Inputs/system-header-simulator.h" int atoi(const char *nptr); Index: test/Analysis/lit.local.cfg =================================================================== --- test/Analysis/lit.local.cfg +++ test/Analysis/lit.local.cfg @@ -14,9 +14,12 @@ config.substitutions.append(('%diff_plist', 'diff -u -w -I "/" -I ".:" -I "version"')) -# Diff command for testing SARIF output to reference output. -config.substitutions.append(('%diff_sarif', - '''diff -U1 -w -I ".*file:.*%basename_t" -I '"version":' -I "2\.0\.0\-csd\.[0-9]*\.beta\."''')) +# Filtering command for testing SARIF output against reference output. +config.substitutions.append(('%normalize_sarif', + "grep -Ev '%s|%s|%s'" % + ('^[[:space:]]*"uri": "file:.*%basename_t"$', + '^[[:space:]]*"version": ".* version .*"$', + '^[[:space:]]*"version": "2\.0\.0-csd\.[0-9]*\.beta\.[0-9-]{10}"$'))) if not config.root.clang_staticanalyzer: config.unsupported = True