llvm-project.git/clang/test/Analysis/method-call-path-notes.cpp, branch main

Reapply r362994 & co "[analyzer][tests] Add normalize_plist to replace diff_plist"

2019-06-11T14:21:32+00:00

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

Revert r362994 & co "[analyzer][tests] Add normalize_plist to replace diff_plist"

2019-06-10T23:25:43+00:00

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

[analyzer][tests] Use normalize_plist in place of diff_plist (`cat` cases)

2019-06-10T22:37:31+00:00

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

[analyzer] [NFC] Reverse the argument order for "diff" in tests

2019-01-10T18:15:44+00:00

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

[Analyzer] Define and use diff_plist in tests, NFC

2018-09-17T10:19:46+00:00

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

[analyzer] [NFC] Change the tests by making the version check more resilient

2018-09-11T18:45:15+00:00

llvm-svn: 341978

Revert "Revert "Revert "Revert "[analyzer] Add coverage information to plist output, update tests""""

2018-09-07T21:58:24+00:00

This reverts commit 2f5d71d9fa135be86bb299e7d773036e50bf1df6.

Hopefully fixing tests on Windows.

llvm-svn: 341719

Revert "Revert "Revert "[analyzer] Add coverage information to plist output, update tests"""

2018-09-07T10:27:16+00:00

Reverts analyzer tests from rL341627 again as they still broke windows buildbots

llvm-svn: 341648

Revert "Revert "[analyzer] Add coverage information to plist output, update tests""

2018-09-07T02:02:35+00:00

This reverts commit a39bcab414dd7ace7e490363ecdf01ecce7743fc.

Reverting the revert, fixing tests.

llvm-svn: 341627

Move test inputs into Inputs directory.

2018-08-14T00:18:48+00:00

We don't need a new ExpectedOutputs/ convention. Expected outputs are
just another form of test input.

llvm-svn: 339634