Skip to content

Commit 4ff43cf

Browse files
committed
Remove nll compare mode.
1 parent 6e4d643 commit 4ff43cf

File tree

4 files changed

+18
-20
lines changed

4 files changed

+18
-20
lines changed

src/tests/compiletest.md

+5-6
Original file line numberDiff line numberDiff line change
@@ -501,23 +501,22 @@ To run the tests in a different mode, you need to pass the `--compare-mode`
501501
CLI flag:
502502

503503
```bash
504-
./x.py test src/test/ui --compare-mode=nll
504+
./x.py test src/test/ui --compare-mode=chalk
505505
```
506506

507507
The possible compare modes are:
508508

509-
* `nll` — Runs in the "true" NLL mode with `-Zborrowck=mir`.
510-
See [UI compare modes](ui.md#compare-modes) for more.
511-
* `polonius` — Runs with Polonius with `-Zpolonius -Zborrowck=mir`, and reuses
512-
the `nll` stderr files.
509+
* `polonius` — Runs with Polonius with `-Zpolonius -Zborrowck=mir`.
513510
* `chalk` — Runs with Chalk with `-Zchalk`.
514511
* `split-dwarf` — Runs with unpacked split-DWARF with `-Csplit-debuginfo=unpacked`.
515512
* `split-dwarf-single` — Runs with packed split-DWARF with `-Csplit-debuginfo=packed`.
516513

514+
See [UI compare modes](ui.md#compare-modes) for more information about how UI
515+
tests support different output for different modes.
516+
517517
In CI, compare modes are only used in one Linux builder, and only with the
518518
following settings:
519519

520-
* `src/test/ui`: Uses `nll` mode.
521520
* `src/test/debuginfo`: Uses `split-dwarf` mode.
522521
This helps ensure that none of the debuginfo tests are affected when
523522
enabling split-DWARF.

src/tests/headers.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ Some examples of `X` in `ignore-X` or `only-X`:
125125
* When [remote testing] is used: `remote`
126126
* When debug-assertions are enabled: `debug`
127127
* When particular debuggers are being tested: `cdb`, `gdb`, `lldb`
128-
* Specific [compare modes]: `compare-mode-nll`, `compare-mode-polonius`,
128+
* Specific [compare modes]: `compare-mode-polonius`,
129129
`compare-mode-chalk`, `compare-mode-split-dwarf`,
130130
`compare-mode-split-dwarf-single`
131131

src/tests/running.md

+5-5
Original file line numberDiff line numberDiff line change
@@ -196,13 +196,13 @@ of the `build` directory from time to time.
196196
## Running tests with different "compare modes"
197197

198198
UI tests may have different output depending on certain "modes" that
199-
the compiler is in. For example, when in "non-lexical lifetimes" (NLL)
200-
mode a test `foo.rs` will first look for expected output in
201-
`foo.nll.stderr`, falling back to the usual `foo.stderr` if not found.
202-
To run the UI test suite in NLL mode, one would use the following:
199+
the compiler is in. For example, when using the Polonius
200+
mode, a test `foo.rs` will first look for expected output in
201+
`foo.polonius.stderr`, falling back to the usual `foo.stderr` if not found.
202+
The following will run the UI test suite in Polonius mode:
203203

204204
```bash
205-
./x.py test src/test/ui --compare-mode=nll
205+
./x.py test src/test/ui --compare-mode=polonius
206206
```
207207

208208
See [Compare modes](compiletest.md#compare-modes) for more details.

src/tests/ui.md

+7-8
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ The general form is:
7474
* `32bit.stderr` — compiler stderr with `stderr-per-bitwidth` header on a 32-bit target
7575

7676
A simple example would be `foo.stderr` next to a `foo.rs` test.
77-
A more complex example would be `foo.my-revision.nll.stderr`.
77+
A more complex example would be `foo.my-revision.polonius.stderr`.
7878

7979
There are several [headers](headers.md) which will change how compiletest will
8080
check for output files:
@@ -496,20 +496,19 @@ In some cases, this might result in different output from the compiler.
496496
To support this, different output files can be saved which contain the
497497
output based on the compare mode.
498498

499-
For example, when in "non-lexical lifetimes" (NLL) mode a test `foo.rs` will
500-
first look for expected output in `foo.nll.stderr`, falling back to the usual
499+
For example, when using the Polonius mode, a test `foo.rs` will
500+
first look for expected output in `foo.polonius.stderr`, falling back to the usual
501501
`foo.stderr` if not found.
502-
This is useful as "true" NLL mode can sometimes result in different
503-
diagnostics and behavior compared to the "migrate mode" NLL (which is the
504-
current default).
502+
This is useful as different modes can sometimes result in different
503+
diagnostics and behavior.
505504
This can help track which tests have differences between the modes, and to
506505
visually inspect those diagnostic differences.
507506

508507
If in the rare case you encounter a test that has different behavior, you can
509508
run something like the following to generate the alternate stderr file:
510509

511510
```sh
512-
./x.py test src/test/ui --compare-mode=nll --bless
511+
./x.py test src/test/ui --compare-mode=polonius --bless
513512
```
514513

515-
Currently, only `nll` mode is checked in CI for UI tests.
514+
Currently none of the compare modes are checked in CI for UI tests.

0 commit comments

Comments
 (0)