]> git.lizzy.rs Git - rust.git/commit
Rollup merge of #67122 - petrochenkov:nodedup, r=estebank
authorYuki Okushi <huyuumi.dev@gmail.com>
Thu, 9 Jan 2020 19:18:30 +0000 (04:18 +0900)
committerGitHub <noreply@github.com>
Thu, 9 Jan 2020 19:18:30 +0000 (04:18 +0900)
commit68ae55bfac1a60fb3c9f45b3a223358ea598f56d
tree0d1ddd542303d2ac2e00dcfe83fc09d18a4eeb92
parent92bd267ba8d8d12cba15490c618628c1f3ea5ac5
parentb82cd9f6391df865551bc6c756cc29a7993e39be
Rollup merge of #67122 - petrochenkov:nodedup, r=estebank

Do not deduplicate diagnostics in UI tests

Error reporting infrastructure deduplicates identical diagnostics with identical spans.

While it's preferable to do this in "release"/"user-facing" mode, it sometimes brings [confusion](https://github.com/rust-lang/rust/pull/50682#issuecomment-390949878) and hides details that may be important during development.

Do we run some passes multiple times when we could do it once?
How many times we run them exactly? Can this number be large? Can the multiplied error construction be expensive? Can speculative checks be made cheaper if they don't report errors?

*Relying* on this mechanism to deduplicate some specific error never looks like a proper solution to me personally.

In this PR I attempt to disable this deduplication by applying `-Z deduplicate-diagnostics=no` to UI tests.