Skip to content

Commit 7ad2d64

Browse files
committed
Add internal_features lint
It lints against features that are inteded to be internal to the compiler and standard library. Implements MCP #596. We allow `internal_features` in the standard library and compiler as those use many features and this _is_ the standard library from the "internal to the compiler and standard library" after all. Marking some features as internal wasn't exactly the most scientific approach, I just marked some mostly obvious features. While there is a categorization in the macro, it's not very well upheld (should probably be fixed in another PR). We always pass `-Ainternal_features` in the testsuite About 400 UI tests and several other tests use internal features. Instead of throwing the attribute on each one, just always allow them. There's nothing wrong with testing internal features^^
1 parent c77d14e commit 7ad2d64

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

tests/compiletest.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ fn test_config(target: &str, path: &str, mode: Mode, with_dependencies: bool) ->
5353
// Add some flags we always want.
5454
program.args.push("-Dwarnings".into());
5555
program.args.push("-Dunused".into());
56+
program.args.push("-Ainternal_features".into());
5657
if let Ok(extra_flags) = env::var("MIRIFLAGS") {
5758
for flag in extra_flags.split_whitespace() {
5859
program.args.push(flag.into());

0 commit comments

Comments
 (0)