Merge "Use more inclusive language."
This commit is contained in:
commit
31e3ec3e10
4 changed files with 6 additions and 10 deletions
|
@ -62,7 +62,7 @@ kBionicSonames: Set[str] = set([
|
|||
])
|
||||
|
||||
# Skip these symbols so the benchmark runs on multiple C libraries (glibc, Bionic, musl).
|
||||
kBionicSymbolBlacklist: Set[str] = set([
|
||||
kBionicIgnoredSymbols: Set[str] = set([
|
||||
'__FD_ISSET_chk',
|
||||
'__FD_SET_chk',
|
||||
'__assert',
|
||||
|
@ -169,7 +169,7 @@ def make_asm_file(lib: LoadedLibrary, is_main: bool, out_filename: Path, map_out
|
|||
nonlocal defs
|
||||
d = defs.get(name)
|
||||
if d is not None and d.soname in kBionicSonames:
|
||||
if name in kBionicSymbolBlacklist: return None
|
||||
if name in kBionicIgnoredSymbols: return None
|
||||
# Discard relocations to newer Bionic symbols, because there aren't many of them, and
|
||||
# they would limit where the benchmark can run.
|
||||
if ver == 'LIBC': return name
|
||||
|
|
|
@ -10,7 +10,7 @@
|
|||
# Profiled execution.
|
||||
profile=no
|
||||
|
||||
# Add files or directories to the blacklist. They should be base names, not
|
||||
# Add files or directories to the ignore list. They should be base names, not
|
||||
# paths.
|
||||
ignore=CVS
|
||||
|
||||
|
|
|
@ -141,7 +141,7 @@ static CompilationRequirements collectRequirements(const Arch& arch,
|
|||
}
|
||||
|
||||
auto new_end = std::remove_if(headers.begin(), headers.end(), [&arch](llvm::StringRef header) {
|
||||
for (const auto& it : header_blacklist) {
|
||||
for (const auto& it : ignored_headers) {
|
||||
if (it.second.find(arch) == it.second.end()) {
|
||||
continue;
|
||||
}
|
||||
|
|
|
@ -33,15 +33,11 @@ extern bool add_include;
|
|||
} \
|
||||
} while (0)
|
||||
|
||||
static const std::unordered_map<std::string, std::set<Arch>> header_blacklist = {
|
||||
static const std::unordered_map<std::string, std::set<Arch>> ignored_headers = {
|
||||
// Internal header.
|
||||
// TODO: we should probably just admit we're never getting rid of this.
|
||||
{ "sys/_system_properties.h", supported_archs },
|
||||
|
||||
// time64.h #errors when included on LP64 archs.
|
||||
{ "time64.h", { Arch::arm64, Arch::x86_64 } },
|
||||
};
|
||||
|
||||
static const std::unordered_set<std::string> missing_symbol_whitelist = {
|
||||
// atexit comes from crtbegin.
|
||||
"atexit",
|
||||
};
|
||||
|
|
Loading…
Reference in a new issue