[PATCH] D55707: Update to SARIF 11-28

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 14 08:26:20 PST 2018


aaron.ballman created this revision.
aaron.ballman added reviewers: NoQ, george.karpenkov.

This updates our SARIF support from 10-10 to 11-28. Functional changes include:

- The run.files property is now an array instead of a mapping.
- fileLocation objects now have a fileIndex property specifying the array index into run.files.
- The resource.rules property is now an array instead of a mapping.
- The result object was given a ruleIndex property that is an index into the resource.rules array.
- rule objects now have their "id" field filled out in addition to the name field.
- Updated the schema and spec version numbers to 11-28.

(Note, the SARIF viewer plugins for Visual Studio and VSCode have not caught up to 11-28 yet, but are expected to be updated.)


https://reviews.llvm.org/D55707

Files:
  lib/StaticAnalyzer/Core/SarifDiagnostics.cpp
  test/Analysis/diagnostics/Inputs/expected-sarif/sarif-diagnostics-taint-test.c.sarif
  test/Analysis/diagnostics/Inputs/expected-sarif/sarif-multi-diagnostic-test.c.sarif

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55707.178236.patch
Type: text/x-patch
Size: 16435 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181214/4dba9dc2/attachment-0001.bin>


More information about the cfe-commits mailing list