<feed xmlns='http://www.w3.org/2005/Atom'>
<title>bcm5719-llvm/clang/test/Analysis/plist-output.m, branch meklort-10.0.0</title>
<subtitle>Project Ortega BCM5719 LLVM</subtitle>
<id>https://git.raptorcs.com/git/bcm5719-llvm/atom?h=meklort-10.0.0</id>
<link rel='self' href='https://git.raptorcs.com/git/bcm5719-llvm/atom?h=meklort-10.0.0'/>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/'/>
<updated>2019-09-06T20:55:29+00:00</updated>
<entry>
<title>[analyzer] Add minimal support for fix-it hints.</title>
<updated>2019-09-06T20:55:29+00:00</updated>
<author>
<name>Artem Dergachev</name>
<email>artem.dergachev@gmail.com</email>
</author>
<published>2019-09-06T20:55:29+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=6cee434ed10ead6b7416ca5ee9592b2b207eeb0f'/>
<id>urn:sha1:6cee434ed10ead6b7416ca5ee9592b2b207eeb0f</id>
<content type='text'>
Allow attaching fixit hints to Static Analyzer BugReports.

Fixits are attached either to the bug report itself or to its notes
(path-sensitive event notes or path-insensitive extra notes).

Add support for fixits in text output (including the default text output that
goes without notes, as long as the fixit "belongs" to the warning).

Add support for fixits in the plist output mode.

Implement a fixit for the path-insensitive DeadStores checker. Only dead
initialization warning is currently covered.

Implement a fixit for the path-sensitive VirtualCall checker when the virtual
method is not pure virtual (in this case the "fix" is to suppress the warning
by qualifying the call).

Both fixits are under an off-by-default flag for now, because they
require more careful testing.

Differential Revision: https://reviews.llvm.org/D65182

llvm-svn: 371257
</content>
</entry>
<entry>
<title>Reapply r362994 &amp; co "[analyzer][tests] Add normalize_plist to replace diff_plist"</title>
<updated>2019-06-11T14:21:32+00:00</updated>
<author>
<name>Hubert Tong</name>
<email>hubert.reinterpretcast@gmail.com</email>
</author>
<published>2019-06-11T14:21:32+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=61c848d2549b5fa4e09776bb0a2d5306e001cb66'/>
<id>urn:sha1:61c848d2549b5fa4e09776bb0a2d5306e001cb66</id>
<content type='text'>
Following r363007, which reverted r362998, r362996, and r362994,
reapply with adjustments for the CRLF differences encountered with
Windows. Namely, the `-b` option of `diff` is employed, and the `grep`
patterns have `$` replaced with `[[:space:]]*$`.

llvm-svn: 363069
</content>
</entry>
<entry>
<title>Revert r362994 &amp; co "[analyzer][tests] Add normalize_plist to replace diff_plist"</title>
<updated>2019-06-10T23:25:43+00:00</updated>
<author>
<name>Reid Kleckner</name>
<email>rnk@google.com</email>
</author>
<published>2019-06-10T23:25:43+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=e7ea1834f467db618a8d8872d7eb9adc14ef5aac'/>
<id>urn:sha1:e7ea1834f467db618a8d8872d7eb9adc14ef5aac</id>
<content type='text'>
Reverts r362998, r362996, and r362994 because the tests do not pass on
Windows due to CRLF changes. Adding back `-w` to diff is not enough, the
new grep substitution doesn't work on Windows, and fixing it is
non-trivial.

llvm-svn: 363007
</content>
</entry>
<entry>
<title>[analyzer][tests] Use normalize_plist in place of diff_plist (`cat` cases)</title>
<updated>2019-06-10T22:37:31+00:00</updated>
<author>
<name>Hubert Tong</name>
<email>hubert.reinterpretcast@gmail.com</email>
</author>
<published>2019-06-10T22:37:31+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=65db5d4cb510d93a9ba1e55b4dbd1325eb41ee74'/>
<id>urn:sha1:65db5d4cb510d93a9ba1e55b4dbd1325eb41ee74</id>
<content type='text'>
Summary:
The `%diff_plist` lit substitution invokes `diff` with a non-portable
`-I` option. The intended effect can be achieved by normalizing the
inputs to `diff` beforehand. Such normalization can be done with
`grep -Ev`, which is also used by other tests.

This patch applies the change (adjusted for review comments) described
in http://lists.llvm.org/pipermail/cfe-dev/2019-April/061904.html
mechanically to the cases where the output file is piped to
`%diff_plist` via `cat`.

The changes were applied via a script, except that
`clang/test/Analysis/NewDelete-path-notes.cpp` and
`clang/test/Analysis/plist-macros-with-expansion.cpp` were each adjusted
for the line-continuation on the relevant `RUN` step.

Reviewers: NoQ, sfertile, xingxue, jasonliu, daltenty

Subscribers: xazax.hun, baloghadamsoftware, szepet, a.sidorin, mikhail.ramalho, Szelethus, donat.nagy, dkrupp, Charusso, jsji, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D62950

llvm-svn: 362996
</content>
</entry>
<entry>
<title>[analyzer] [NFC] Reverse the argument order for "diff" in tests</title>
<updated>2019-01-10T18:15:44+00:00</updated>
<author>
<name>George Karpenkov</name>
<email>ekarpenkov@apple.com</email>
</author>
<published>2019-01-10T18:15:44+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=363dd8e4b86cab350999afbbaf23c5f8fa472b47'/>
<id>urn:sha1:363dd8e4b86cab350999afbbaf23c5f8fa472b47</id>
<content type='text'>
The current argument order has "expected" and "actual" the wrong way around,
so that the diff shows the change from expected to actual, not from actual to expected.

Namely, if the expected diagnostics contains the string "foo", but the analyzer emits "bar",
we really want to see:

```
- foo
+ bar
```

not

```
- bar
+ foo
```

since adapting to most changes would require applying that diff to the expected output.

Differential Revision: https://reviews.llvm.org/D56340

llvm-svn: 350866
</content>
</entry>
<entry>
<title>[Analyzer] Define and use diff_plist in tests, NFC</title>
<updated>2018-09-17T10:19:46+00:00</updated>
<author>
<name>Mikhail Maltsev</name>
<email>mikhail.maltsev@arm.com</email>
</author>
<published>2018-09-17T10:19:46+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=c704f4d5615ba512aafe85e7b2898d5335608d3e'/>
<id>urn:sha1:c704f4d5615ba512aafe85e7b2898d5335608d3e</id>
<content type='text'>
This patch defines a new substitution and uses it to reduce
duplication in the Clang Analyzer test cases.

Differential Revision: https://reviews.llvm.org/D52036

llvm-svn: 342365
</content>
</entry>
<entry>
<title>[analyzer] [NFC] Change the tests by making the version check more resilient</title>
<updated>2018-09-11T18:45:15+00:00</updated>
<author>
<name>George Karpenkov</name>
<email>ekarpenkov@apple.com</email>
</author>
<published>2018-09-11T18:45:15+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=5594866a64587e716922f894c7a0dac0ef857c10'/>
<id>urn:sha1:5594866a64587e716922f894c7a0dac0ef857c10</id>
<content type='text'>
llvm-svn: 341978
</content>
</entry>
<entry>
<title>Revert "Revert "Revert "Revert "[analyzer] Add coverage information to plist output, update tests""""</title>
<updated>2018-09-07T21:58:24+00:00</updated>
<author>
<name>George Karpenkov</name>
<email>ekarpenkov@apple.com</email>
</author>
<published>2018-09-07T21:58:24+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=8a88c835931e74ea282b40c7a59630b749e08d83'/>
<id>urn:sha1:8a88c835931e74ea282b40c7a59630b749e08d83</id>
<content type='text'>
This reverts commit 2f5d71d9fa135be86bb299e7d773036e50bf1df6.

Hopefully fixing tests on Windows.

llvm-svn: 341719
</content>
</entry>
<entry>
<title>Revert "Revert "Revert "[analyzer] Add coverage information to plist output, update tests"""</title>
<updated>2018-09-07T10:27:16+00:00</updated>
<author>
<name>Simon Pilgrim</name>
<email>llvm-dev@redking.me.uk</email>
</author>
<published>2018-09-07T10:27:16+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=b746df0b0a1637803b131598daca5907955facb3'/>
<id>urn:sha1:b746df0b0a1637803b131598daca5907955facb3</id>
<content type='text'>
Reverts analyzer tests from rL341627 again as they still broke windows buildbots

llvm-svn: 341648
</content>
</entry>
<entry>
<title>Revert "Revert "[analyzer] Add coverage information to plist output, update tests""</title>
<updated>2018-09-07T02:02:35+00:00</updated>
<author>
<name>George Karpenkov</name>
<email>ekarpenkov@apple.com</email>
</author>
<published>2018-09-07T02:02:35+00:00</published>
<link rel='alternate' type='text/html' href='https://git.raptorcs.com/git/bcm5719-llvm/commit/?id=d7acacab94f3eac228c7b66a0c76972c7930dc9e'/>
<id>urn:sha1:d7acacab94f3eac228c7b66a0c76972c7930dc9e</id>
<content type='text'>
This reverts commit a39bcab414dd7ace7e490363ecdf01ecce7743fc.

Reverting the revert, fixing tests.

llvm-svn: 341627
</content>
</entry>
</feed>
