RepoMirrors/libabigail
1
0
Fork 0
mirror of git://sourceware.org/git/libabigail.git synced 2025-03-31 16:00:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add a --no-architecture option to abidiff

Browse Source 
This new option omits architectures when comparing ABIs.

	* tools/abidiff.cc (options::no_arch): New data member.
	(options::options): Initialize it.
	(display_usage): Display a help string for the new options.
	(parse_command_line): Parse the new options.
	(main): If --no-architecture is provided, set the corpus
	architecture to "".
	* doc/manuals/abidiff.rst: Document the new options.

Signed-off-by: Dodji Seketeli <dodji@redhat.com>
This commit is contained in:
Dodji Seketeli 2015-08-29 12:59:13 +02:00
parent 843de38b6c
commit e61afa7291
2 changed files with 29 additions and 12 deletions
doc/manuals
abidiff.rst
tools
abidiff.cc

4
doc/manuals/abidiff.rst
View File

@ -198,6 +198,10 @@ Options
redundant change is a change that has been displayed elsewhere in
the report. This option is switched on by default.
* ``--no-architecture``
Do not take architecture in account when comparing ABIs.
* ``--dump-diff-tree``
After the diff report, emit a textual representation of the diff

37
tools/abidiff.cc
View File

@ -63,6 +63,7 @@ struct options
vector<string> drop_var_regex_patterns;
vector<string> keep_fn_regex_patterns;
vector<string> keep_var_regex_patterns;
bool no_arch;
bool show_stats_only;
bool show_symtabs;
bool show_deleted_fns;
@ -84,22 +85,23 @@ struct options
shared_ptr<char> di_root_path2;
options()
: display_usage(false),
missing_operand(false),
show_stats_only(false),
show_symtabs(false),
show_deleted_fns(false),
show_changed_fns(false),
show_added_fns(false),
: display_usage(),
missing_operand(),
no_arch(),
show_stats_only(),
show_symtabs(),
show_deleted_fns(),
show_changed_fns(),
show_added_fns(),
show_all_fns(true),
show_deleted_vars(false),
show_changed_vars(false),
show_added_vars(false),
show_deleted_vars(),
show_changed_vars(),
show_added_vars(),
show_all_vars(true),
show_linkage_names(true),
show_harmful_changes(true),
show_harmless_changes(false),
show_redundant_changes(false),
show_harmless_changes(),
show_redundant_changes(),
show_symbols_not_referenced_by_debug_info(true),
dump_diff_tree(),
show_stats()
@ -116,6 +118,7 @@ display_usage(const string& prog_name, ostream& out)
<< " --help|-h display this message\n "
<< " --stat only display the diff stats\n"
<< " --symtabs only display the symbol tables of the corpora\n"
<< " --no-architecture do not take architecture in account\n"
<< " --deleted-fns display deleted public functions\n"
<< " --changed-fns display changed public functions\n"
<< " --added-fns display added public functions\n"
@ -212,6 +215,8 @@ parse_command_line(int argc, char* argv[], options& opts)
opts.display_usage = true;
return true;
}
else if (!strcmp(argv[i], "--no-architecture"))
opts.no_arch = true;
else if (!strcmp(argv[i], "--deleted-fns"))
{
opts.show_deleted_fns = true;
@ -668,6 +673,14 @@ main(int argc, char* argv[])
return abigail::tools_utils::ABIDIFF_ERROR;
}
if (opts.no_arch)
{
if (c1)
c1->set_architecture_name("");
if (c2)
c2->set_architecture_name("");
}
if (t1)
{
translation_unit_diff_sptr diff = compute_diff(t1, t2);