LineageOS/android_development
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

header-abi-diff: Add a flag to consider opaque types with different names structurally unequal.

Browse Source 
Bug: 79576032

Test: tests/test.py

Change-Id: I28cb7a2676a1e85e4ef1d2faaf6684d7c1824370
This commit is contained in:
Jayant Chowdhary
2018-05-31 15:31:32 -07:00
parent 4cb768402c
commit e90b33a9d9
10 changed files with 84 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
vndk/tools/header-checker/header-abi-diff/src/header_abi_diff.cpp
View File

@@ -85,6 +85,14 @@ static llvm::cl::opt<bool> allow_unreferenced_changes(
" APIs."),
llvm::cl::Optional, llvm::cl::cat(header_checker_category));
static llvm::cl::opt<bool> consider_opaque_types_different(
"consider-opaque-types-different",
llvm::cl::desc("Consider opaque types with different names as different"
" .This should not be used while comparing C++ library"
" ABIs"),
llvm::cl::Optional, llvm::cl::cat(header_checker_category));
static llvm::cl::opt<abi_util::TextFormatIR> text_format_old(
"text-format-old", llvm::cl::desc("Specify text format of old abi dump"),
llvm::cl::values(clEnumValN(abi_util::TextFormatIR::ProtobufTextFormat,
@@ -142,9 +150,11 @@ int main(int argc, const char **argv) {
if (llvm::sys::fs::exists(ignore_symbol_list)) {
ignored_symbols = LoadIgnoredSymbols(ignore_symbol_list);
}
abi_util::DiffPolicyOptions diff_policy_options(
consider_opaque_types_different);
HeaderAbiDiff judge(lib_name, arch, old_dump, new_dump, compatibility_report,
ignored_symbols, check_all_apis, text_format_old,
text_format_new, text_format_diff);
ignored_symbols, diff_policy_options, check_all_apis,
text_format_old, text_format_new, text_format_diff);
abi_util::CompatibilityStatusIR status = judge.GenerateCompatibilityReport();