squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

implement "only-<platforms>" for test headers

Browse source 
This patch implements "only-<platforms>" for tests headers using which one can
specify just the platforms on which the test should run rather than listing all
the platforms to ignore using "ignore-<platforms>".

This is a fix for issues #33581 and #47459.
This commit is contained in:
Pulkit Goyal 2018-01-16 14:37:05 +05:30
parent 79a521bb9a
commit fd075c6f21
1 changed files with 13 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/tools/compiletest/src/header.rs
View file

@ -44,6 +44,8 @@ impl EarlyProps {
props.ignore =
props.ignore ||
config.parse_cfg_name_directive(ln, "ignore") ||
(config.parse_cfg_prefix(ln, "only") &&
!config.parse_cfg_name_directive(ln, "only")) ||
ignore_gdb(config, ln) ||
ignore_lldb(config, ln) ||
ignore_llvm(config, ln);
@ -564,6 +566,17 @@ impl Config {
}
}
fn parse_cfg_prefix(&self, line: &str, prefix: &str) -> bool {
// returns whether this line contains this prefix or not. For prefix
// "ignore", returns true if line says "ignore-x86_64", "ignore-arch",
// "ignore-andorid" etc.
if line.starts_with(prefix) && line.as_bytes().get(prefix.len()) == Some(&b'-') {
true
} else {
false
}
}
fn parse_name_directive(&self, line: &str, directive: &str) -> bool {
// Ensure the directive is a whole word. Do not match "ignore-x86" when
// the line says "ignore-x86_64".