Skip to content

compiletest: Add directives to detect sanitizer support - #73044

Merged
bors merged 1 commit into
rust-lang:masterfrom
tmiasko:compiletest-san
Jun 19, 2020
Merged

compiletest: Add directives to detect sanitizer support#73044
bors merged 1 commit into
rust-lang:masterfrom
tmiasko:compiletest-san

Conversation

@tmiasko

@tmiaskotmiasko commented Jun 5, 2020

Copy link
Copy Markdown
Contributor

Add needs-sanitizer-{address,leak,memory,thread} directive indicating
that test requires target with support for specific sanitizer.

This is an addition to the existing needs-sanitizer-support directive
indicating that test requires a sanitizer runtime library.

The existing needs-sanitizer-support directive could be incorporated into the
new ones, but I decided to retain it, since it enables running sanitizer
codegen tests even when building of sanitizer runtime libraries is disabled.

@rust-highfive

rust-highfive commented Jun 5, 2020

Copy link
Copy Markdown
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nikomatsakis (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 5, 2020
@tmiaskotmiasko mentioned this pull request Jun 6, 2020
Merged
@nikomatsakis

nikomatsakis commented Jun 8, 2020

Copy link
Copy Markdown
Contributor

This is great. It'd be nice to see it documented in rustc-dev-guide, but the documentation there seems sufficiently sparse that it's not obvious where you would add it. I'm going to r+ but if you want to go and update the rustc-dev-guide coverage of these sorts of flags, it would be a heroic action on your part, @tmiasko =)

@bors r+

@bors

bors commented Jun 8, 2020

Copy link
Copy Markdown
Collaborator

📌 Commit bae11ff has been approved by nikomatsakis

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 8, 2020
@tmiaskotmiasko mentioned this pull request Jun 8, 2020
Merged
@tmiasko

tmiasko commented Jun 10, 2020

Copy link
Copy Markdown
ContributorAuthor

Updated annotations in two tests missed previously src/test/ui/sanitize/{leak,memory}.rs.

@nikomatsakis

nikomatsakis commented Jun 11, 2020

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Jun 11, 2020

Copy link
Copy Markdown
Collaborator

📌 Commit 975f7df has been approved by nikomatsakis

Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Jun 13, 2020
@Dylan-DPC
Rollup merge ofrust-lang#73044- tmiasko:compiletest-san, r=nikomats…
9163d1b
…akis
compiletest: Add directives to detect sanitizer support
Add needs-sanitizer-{address,leak,memory,thread} directive indicating
that test requires target with support for specific sanitizer.
This is an addition to the existing needs-sanitizer-support directive
indicating that test requires a sanitizer runtime library.
The existing needs-sanitizer-support directive could be incorporated into the
new ones, but I decided to retain it, since it enables running sanitizer
codegen tests even when building of sanitizer runtime libraries is disabled.
@Dylan-DPC-zzDylan-DPC-zz mentioned this pull request Jun 13, 2020
Closed
RalfJung added a commit to RalfJung/rust that referenced this pull request Jun 13, 2020
@RalfJung
Rollup merge ofrust-lang#73044- tmiasko:compiletest-san, r=nikomats…
de144aa
…akis
compiletest: Add directives to detect sanitizer support
Add needs-sanitizer-{address,leak,memory,thread} directive indicating
that test requires target with support for specific sanitizer.
This is an addition to the existing needs-sanitizer-support directive
indicating that test requires a sanitizer runtime library.
The existing needs-sanitizer-support directive could be incorporated into the
new ones, but I decided to retain it, since it enables running sanitizer
codegen tests even when building of sanitizer runtime libraries is disabled.
@RalfJungRalfJung mentioned this pull request Jun 13, 2020
Closed
RalfJung added a commit to RalfJung/rust that referenced this pull request Jun 13, 2020
@RalfJung
Rollup merge ofrust-lang#73044- tmiasko:compiletest-san, r=nikomats…
cbbc200
…akis
compiletest: Add directives to detect sanitizer support
Add needs-sanitizer-{address,leak,memory,thread} directive indicating
that test requires target with support for specific sanitizer.
This is an addition to the existing needs-sanitizer-support directive
indicating that test requires a sanitizer runtime library.
The existing needs-sanitizer-support directive could be incorporated into the
new ones, but I decided to retain it, since it enables running sanitizer
codegen tests even when building of sanitizer runtime libraries is disabled.
@RalfJungRalfJung mentioned this pull request Jun 13, 2020
Closed
RalfJung added a commit to RalfJung/rust that referenced this pull request Jun 13, 2020
@RalfJung
Rollup merge ofrust-lang#73044- tmiasko:compiletest-san, r=nikomats…
4dbb652
…akis
compiletest: Add directives to detect sanitizer support
Add needs-sanitizer-{address,leak,memory,thread} directive indicating
that test requires target with support for specific sanitizer.
This is an addition to the existing needs-sanitizer-support directive
indicating that test requires a sanitizer runtime library.
The existing needs-sanitizer-support directive could be incorporated into the
new ones, but I decided to retain it, since it enables running sanitizer
codegen tests even when building of sanitizer runtime libraries is disabled.
@RalfJungRalfJung mentioned this pull request Jun 13, 2020
Closed
@tmiasko

tmiasko commented Jun 13, 2020

Copy link
Copy Markdown
ContributorAuthor
Failed on x86_64-apple-darwin in rollup #73299
[run-make] run-make-fulldeps/sanitizer-cdylib-link
[run-make] run-make-fulldeps/sanitizer-dylib-link
error: linking with `cc` failed: exit code: 1 ...

@RalfJung

RalfJung commented Jun 13, 2020

Copy link
Copy Markdown
Member

@bors r-

@borsbors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jun 13, 2020
@tmiasko
compiletest: Add directives to detect sanitizer support
d40e624
Add needs-sanitizer-{address,leak,memory,thread} directive indicating
that test requires target with support for specific sanitizer.
This is an addition to the existing needs-sanitizer-support directive
indicating that test requires a sanitizer runtime library.
@tmiasko

tmiasko commented Jun 13, 2020

Copy link
Copy Markdown
ContributorAuthor

Retained only-linux annotations on sanitizer-*-link tests, since they don't work on macOS.

@nikomatsakis

nikomatsakis commented Jun 15, 2020

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Jun 15, 2020

Copy link
Copy Markdown
Collaborator

📌 Commit d40e624 has been approved by nikomatsakis

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jun 15, 2020
@RalfJungRalfJung mentioned this pull request Jun 19, 2020
Merged
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 19, 2020
@bors
Auto merge ofrust-lang#73504- RalfJung:rollup-iy8hsvl, r=RalfJung
72417d8
Rollup of 10 pull requests
Successful merges:
- rust-lang#72280 (Fix up autoderef when reborrowing)
- rust-lang#72785 (linker: MSVC supports linking static libraries as a whole archive)
- rust-lang#73011 (first stage of implementing LLVM code coverage)
- rust-lang#73044 (compiletest: Add directives to detect sanitizer support)
- rust-lang#73054 (memory access sanity checks: abort instead of panic)
- rust-lang#73136 (Change how compiler-builtins gets many CGUs)
- rust-lang#73280 (Add E0763)
- rust-lang#73317 (bootstrap: read config from $RUST_BOOTSTRAP_CONFIG)
- rust-lang#73350 (bootstrap/install.rs: support a nonexistent `prefix` in `x.py install`)
- rust-lang#73352 (Speed up bootstrap a little.)
Failed merges:
r? @ghost
@bors
bors merged commit 0851036 into rust-lang:masterJun 19, 2020
@tmiasko
tmiasko deleted the compiletest-san branch July 1, 2020 13:47
@cuvipercuviper added this to the 1.46 milestone May 2, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@nikomatsakisnikomatsakis

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Milestone

1.46

Development

Successfully merging this pull request may close these issues.

6 participants

@tmiasko@rust-highfive@nikomatsakis@bors@RalfJung@cuviper