From patchwork Wed Oct 4 10:52:41 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dodji Seketeli X-Patchwork-Id: 77090 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 21B753858CDB for ; Wed, 4 Oct 2023 10:52:52 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 21B753858CDB DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1696416772; bh=Jh8JJqAdsj/RCBNqhA4dP0ZtsUQkAcGjEWnWe3mhtqI=; h=To:Subject:References:Date:In-Reply-To:List-Id:List-Unsubscribe: List-Archive:List-Help:List-Subscribe:From:Reply-To:Cc:From; b=dAvktdK2oO1WxWQc8GliRDswWe/ifYe/bvACxJUYSfXW/F/p+/ZtiJ0UMqjti2clz t7Hf9x7Evozxt7P1qCe8L2EAcMNsADypNMoPM4lUp3MWo63AezFUhyPSJqiWbaD394 XH0C95LZhqjIhyHZv9hXb05xhNFBnfQFIWlzV2U8= X-Original-To: libabigail@sourceware.org Delivered-To: libabigail@sourceware.org Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTPS id 14386385841B for ; Wed, 4 Oct 2023 10:52:47 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 14386385841B Received: from mail-qk1-f200.google.com (mail-qk1-f200.google.com [209.85.222.200]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-499-zG5gFBM3MwysP4POcoX20A-1; Wed, 04 Oct 2023 06:52:45 -0400 X-MC-Unique: zG5gFBM3MwysP4POcoX20A-1 Received: by mail-qk1-f200.google.com with SMTP id af79cd13be357-7740c0e88ffso231835285a.1 for ; Wed, 04 Oct 2023 03:52:45 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1696416765; x=1697021565; h=mime-version:user-agent:message-id:in-reply-to:date:references :organization:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=Jh8JJqAdsj/RCBNqhA4dP0ZtsUQkAcGjEWnWe3mhtqI=; b=wf2fJ//gW8IDWN+bBxmhrXkcuMgxDydK1RsU6J7y7niVEnH6+VjuzrLIMw2kGhYLyW z0p1iIVDjuV11EHjBvIWTThnwZLY3SfqclEHVVRue5dxuU9p7M5JNoKD6oE20ox5+tdh TQq801UUvljotXMuAvKH3Facs+Vw3lvNd0lvSwm0T57cqkRpCqPS6Xmfr8PCDRiFDICb I52E1fIDduhGj3XhMyr5GRoT3A2THeTKM5Nv01yr4+1+h6VtSeKs+zFrp5L2hKSzuAmN xQwwaUy2P2B/GBbV58lsPdTU7dBWcbE0f9wVXDR0HJij+O+fvBa6NLRE5SmymWokk8RP ir/A== X-Gm-Message-State: AOJu0YxX/PIhurRNJP5HhGg152Zg5ipjXXNXfkhy/I/ovQPK6MVj7QfS jqta5u2pItm5c5uY8FElyHuJpT3kXaAgG8XLDLmsIx2nMaFcrGK9HErAY5P4lXfIbJjFmLQ7Pg4 qXR5CE/NcC7T5Rb50NYrQ X-Received: by 2002:a05:620a:21cc:b0:76f:1e31:93cd with SMTP id h12-20020a05620a21cc00b0076f1e3193cdmr2044289qka.43.1696416765021; Wed, 04 Oct 2023 03:52:45 -0700 (PDT) X-Google-Smtp-Source: AGHT+IGcBJ/0bWrgStkFfzeZXMRdvPVbGlIaaDi7sWA2ZDkiRz4a+SGxQV7YTBng277zxvj2nVhG2w== X-Received: by 2002:a05:620a:21cc:b0:76f:1e31:93cd with SMTP id h12-20020a05620a21cc00b0076f1e3193cdmr2044279qka.43.1696416764773; Wed, 04 Oct 2023 03:52:44 -0700 (PDT) Received: from localhost ([88.120.130.27]) by smtp.gmail.com with ESMTPSA id f22-20020a05620a12f600b00767c961eb47sm1167125qkl.43.2023.10.04.03.52.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 04 Oct 2023 03:52:44 -0700 (PDT) Received: by localhost (Postfix, from userid 1000) id 03CB1B6EB1; Wed, 4 Oct 2023 12:52:41 +0200 (CEST) To: libabigail@sourceware.org Subject: [PATCH 1/3, V2] For WIP branch check-uapi-support: default-reporter,reporter-priv: Do not report names of anonymous enums Organization: Red Hat / France References: <87r0mapspt.fsf@redhat.com> X-Operating-System: CentOS Stream release 9 X-URL: http://www.redhat.com Date: Wed, 04 Oct 2023 12:52:41 +0200 In-Reply-To: <87r0mapspt.fsf@redhat.com> (Dodji Seketeli's message of "Wed, 04 Oct 2023 12:50:06 +0200") Message-ID: <87mswypsli.fsf@redhat.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Spam-Status: No, score=-11.9 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H4, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: libabigail@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Mailing list of the Libabigail project List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-Patchwork-Original-From: Dodji Seketeli via Libabigail From: Dodji Seketeli Reply-To: Dodji Seketeli Cc: Trilok Soni , dodji@redhat.com, Satya Durga Srinivasu Prabhala Errors-To: libabigail-bounces+patchwork=sourceware.org@sourceware.org Sender: "Libabigail" Hello, When reporting changes of anonymous enums, do not try to report their qualified name as that doesn't make any sense -- they are anonymous. Similarly, in report_name_size_and_alignment_changes do not try to report about changes in the name of an anonymous enum. * src/abg-default-reporter.cc (default_reporter::report): In the overload for enum_diff, do not get the qualified name of anonymous enums. * src/abg-reporter-priv.cc (report_name_size_and_alignment_changes): Do not report about name changes for anonymous enums. Signed-off-by: Dodji Seketeli --- src/abg-default-reporter.cc | 12 +++++++++--- src/abg-reporter-priv.cc | 3 ++- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/src/abg-default-reporter.cc b/src/abg-default-reporter.cc index b1df9300..c71f8d56 100644 --- a/src/abg-default-reporter.cc +++ b/src/abg-default-reporter.cc @@ -162,7 +162,9 @@ default_reporter::report(const enum_diff& d, ostream& out, { out << indent << " '" - << i->get_qualified_name() + << (first->get_is_anonymous() + ? i->get_name() + : i->get_qualified_name()) << "' value '" << i->get_value() << "'"; @@ -181,7 +183,9 @@ default_reporter::report(const enum_diff& d, ostream& out, { out << indent << " '" - << i->get_qualified_name() + << (second->get_is_anonymous() + ? i->get_name() + :i->get_qualified_name()) << "' value '" << i->get_value() << "'"; @@ -201,7 +205,9 @@ default_reporter::report(const enum_diff& d, ostream& out, { out << indent << " '" - << i->first.get_qualified_name() + << (first->get_is_anonymous() + ? i->first.get_name() + : i->first.get_qualified_name()) << "' from value '" << i->first.get_value() << "' to '" << i->second.get_value() << "'"; diff --git a/src/abg-reporter-priv.cc b/src/abg-reporter-priv.cc index 63a45b80..cc38f240 100644 --- a/src/abg-reporter-priv.cc +++ b/src/abg-reporter-priv.cc @@ -965,7 +965,8 @@ report_name_size_and_alignment_changes(decl_base_sptr first, string fn = first->get_qualified_name(), sn = second->get_qualified_name(); - if (fn != sn) + if (!(first->get_is_anonymous() && second->get_is_anonymous()) + && fn != sn) { if (!(ctxt->get_allowed_category() & HARMLESS_DECL_NAME_CHANGE_CATEGORY) && filtering::has_harmless_name_change(first, second)) From patchwork Wed Oct 4 10:54:12 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dodji Seketeli X-Patchwork-Id: 77092 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id E1CF4385841B for ; Wed, 4 Oct 2023 10:55:51 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org E1CF4385841B DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1696416952; bh=uyZvaSqZuZbPZCzIfOyRTXPOb4TNqJcTmpmP8/Xgw8g=; h=To:Subject:References:Date:In-Reply-To:List-Id:List-Unsubscribe: List-Archive:List-Help:List-Subscribe:From:Reply-To:Cc:From; b=dvif705Afdt7H3YjmSnMq0W0jHz2A80tda4Z/W9fEuWpXtpVuetz2RlxUK3CXVBQw TWiXCZrOmcax6Wyg44gxy615jSK+YhNBDBEzNBxAE45PDP+bdG/C8OQ480OFS4SRGx tqFJN/6GxiiPmNGxwykKbicbSJvjbQ/UwJEAOLi8= X-Original-To: libabigail@sourceware.org Delivered-To: libabigail@sourceware.org Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by sourceware.org (Postfix) with ESMTPS id 5B2673858C52 for ; Wed, 4 Oct 2023 10:54:58 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 5B2673858C52 Received: from mail-qt1-f200.google.com (mail-qt1-f200.google.com [209.85.160.200]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-404-ffhlPwNxP5uHBWUOAXFheQ-1; Wed, 04 Oct 2023 06:54:19 -0400 X-MC-Unique: ffhlPwNxP5uHBWUOAXFheQ-1 Received: by mail-qt1-f200.google.com with SMTP id d75a77b69052e-4194d8b6fceso24359991cf.1 for ; Wed, 04 Oct 2023 03:54:19 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1696416859; x=1697021659; h=mime-version:user-agent:message-id:in-reply-to:date:references :organization:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=uyZvaSqZuZbPZCzIfOyRTXPOb4TNqJcTmpmP8/Xgw8g=; b=AXwfqApfvUJyp9woHTDJAfBwLRKOb7DqgEQFJ9Y8wW+B/SkXbKTlkQRqzq9PVd0YtE 2Fro5pKpZtpRRFhwu1iHLfjHPQZBWuntf2QMIFY6HwUBIygOjl2Mk6BqCM36Dmaxzvwj bwUfWjxNkjOSqbGSEGs2jbnciAnhS9ZkQBaPMzc4MGCwfCit4fDd1GUKeT86xqfVEFyO 2PEtV44oiPEGdS3AUgVZG747Nrek9WpuRJOWcxZRwl9XGQFQkbTTVXA50KNTknjg7k4I Xnk8jpLagJ+iv1C1bNCcd/arglUBgHOliQ/rDtTJxfquah+1HgF0yGyJDTwd/8hWTVjG /DWA== X-Gm-Message-State: AOJu0YwZIOkSio4bxDaZPyRMcoGcce2NDPmjLzWj30bddte3Q5zFgt51 CLj6mYCRQqtOoY2tL8OEaB3vBV+OQzABpe+A89PTaySNoxXf08c0nJ9vF231lkXwR/jMPpPGprZ yLBxaBAXo7sL3q1ZiQxJk X-Received: by 2002:ac8:7f86:0:b0:418:111e:7910 with SMTP id z6-20020ac87f86000000b00418111e7910mr2010121qtj.43.1696416858108; Wed, 04 Oct 2023 03:54:18 -0700 (PDT) X-Google-Smtp-Source: AGHT+IE7KVsLKd0TyIL3J0OMiwTCzWe4GqjnkDpEN6eSFD6gDfWpAUIUWwspMD8MX0d5i0RioCIDvw== X-Received: by 2002:ac8:7f86:0:b0:418:111e:7910 with SMTP id z6-20020ac87f86000000b00418111e7910mr2010091qtj.43.1696416857074; Wed, 04 Oct 2023 03:54:17 -0700 (PDT) Received: from localhost ([88.120.130.27]) by smtp.gmail.com with ESMTPSA id h25-20020ac87159000000b0040331a24f16sm1140605qtp.3.2023.10.04.03.54.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 04 Oct 2023 03:54:15 -0700 (PDT) Received: by localhost (Postfix, from userid 1000) id 65F66B6EB1; Wed, 4 Oct 2023 12:54:12 +0200 (CEST) To: Dodji Seketeli Subject: [PATCH 2/3, V2] For WIP branch check-uapi-support: ir,comparison,corpus: Better support anonymous enums comparison Organization: Red Hat / France References: <87r0mapspt.fsf@redhat.com> X-Operating-System: CentOS Stream release 9 X-URL: http://www.redhat.com Date: Wed, 04 Oct 2023 12:54:12 +0200 In-Reply-To: <87r0mapspt.fsf@redhat.com> (Dodji Seketeli's message of "Wed, 04 Oct 2023 12:50:06 +0200") Message-ID: <87h6n6psiz.fsf@redhat.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Spam-Status: No, score=-11.9 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: libabigail@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Mailing list of the Libabigail project List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-Patchwork-Original-From: Dodji Seketeli via Libabigail From: Dodji Seketeli Reply-To: Dodji Seketeli Cc: Trilok Soni , libabigail@sourceware.org, Satya Durga Srinivasu Prabhala Errors-To: libabigail-bounces+patchwork=sourceware.org@sourceware.org Sender: "Libabigail" Hello, Anonymous enums are designated using their internal names which are automatically generated. The problem however is that whenever a new anonymous enum gets inserted in the translation unit, that can randomly change the name of existing anonymous enums. This makes reporting of changes in anonymous enums essentially useless. To address this issue, this patch uses flat representation for anonymous enums, for the purpose of designating them outside of the context of type canonicalization. This is like what is done already for anonymous classes and unions. * include/abg-fwd.h (get_enum_flat_representation) (get_class_or_enum_flat_representation): Declare new functions. * include/abg-ir.h (enum_type_decl::get_sorted_enumerators): Declare new member functions. * src/abg-comparison.cc (corpus_diff::priv::{ensure_lookup_tables_populated, added_unreachable_type_is_suppressed, deleted_unreachable_type_is_suppressed, record_type_as_reachable_from_public_interfaces}): Use non-internal pretty representation to designate ABI artifacts so that anonymous enums get designated by their flat representation. * src/abg-corpus.cc (corpus::type_is_reachable_from_public_interfaces): Likewise. * src/abg-ir.cc (get_type_name): The type name of an anonymous enum or class should is now the flat representation if we are not in the context of type canonicalization. (enum_type_decl::get_pretty_representation): If we are not in the context of type canonicalization, the pretty representation of an enum is its flat representation. (get_enum_flat_representation) (get_class_or_enum_flat_representation): Define new functions. (enum_type_decl::get_sorted_enumerators): Likewise. * tests/data/test-annotate/libtest23.so.abi: Adjust. * tests/data/test-annotate/test-anonymous-members-0.o.abi: Likewise. * tests/data/test-annotate/test15-pr18892.so.abi: Likewise. * tests/data/test-annotate/test18-pr19037-libvtkRenderingLIC-6.1.so.abi: Likewise. * tests/data/test-annotate/test19-pr19023-libtcmalloc_and_profiler.so.abi: Likewise. * tests/data/test-annotate/test20-pr19025-libvtkParallelCore-6.1.so.abi: Likewise. * tests/data/test-diff-dwarf/test43-PR22913-report-0.txt: Likewise. * tests/data/test-read-dwarf/PR22122-libftdc.so.abi: Likewise. * tests/data/test-read-dwarf/libtest23.so.abi: Likewise. * tests/data/test-read-dwarf/test-libandroid.so.abi: Likewise. * tests/data/test-read-dwarf/test16-pr18904.so.abi: Likewise. * tests/data/test-read-dwarf/test18-pr19037-libvtkRenderingLIC-6.1.so.abi: Likewise. * tests/data/test-read-dwarf/test22-pr19097-libstdc++.so.6.0.17.so.abi: Likewise. Signed-off-by: Dodji Seketeli --- include/abg-fwd.h | 27 + include/abg-ir.h | 3 + src/abg-comparison.cc | 10 +- src/abg-corpus.cc | 4 +- src/abg-ir.cc | 203 +- tests/data/test-annotate/libtest23.so.abi | 224 +- .../test-anonymous-members-0.o.abi | 22 +- .../data/test-annotate/test15-pr18892.so.abi | 8 +- ...st18-pr19037-libvtkRenderingLIC-6.1.so.abi | 54 +- ...19-pr19023-libtcmalloc_and_profiler.so.abi | 10 +- ...st20-pr19025-libvtkParallelCore-6.1.so.abi | 2 +- .../test43-PR22913-report-0.txt | 4 +- .../test-read-dwarf/PR22122-libftdc.so.abi | 12416 ++++++++-------- tests/data/test-read-dwarf/libtest23.so.abi | 216 +- .../test-read-dwarf/test-libandroid.so.abi | 44 +- .../test-read-dwarf/test16-pr18904.so.abi | 1816 +-- ...st18-pr19037-libvtkRenderingLIC-6.1.so.abi | 40 +- .../test22-pr19097-libstdc++.so.6.0.17.so.abi | 60 +- 18 files changed, 7637 insertions(+), 7526 deletions(-) diff --git a/include/abg-fwd.h b/include/abg-fwd.h index 0afb2122..e494de07 100644 --- a/include/abg-fwd.h +++ b/include/abg-fwd.h @@ -1097,6 +1097,33 @@ get_class_or_union_flat_representation(const class_or_union_sptr& cou, bool internal, bool qualified_name = true); +string +get_enum_flat_representation(const enum_type_decl& enum_type, + const string& indent, + bool one_line, + bool internal, + bool qualified_names); + +string +get_enum_flat_representation(const enum_type_decl* enum_type, + const string& indent, + bool one_line, + bool internal, + bool qualified_names); + +string +get_enum_flat_representation(const enum_type_decl_sptr& enum_type, + const string& indent, + bool one_line, + bool qualified_names); + +string +get_class_or_enum_flat_representation(const type_base& coe, + const string& indent, + bool one_line, + bool internal, + bool qualified_name); + string get_debug_representation(const type_or_decl_base*); diff --git a/include/abg-ir.h b/include/abg-ir.h index 18e95afd..3e2cbda9 100644 --- a/include/abg-ir.h +++ b/include/abg-ir.h @@ -2743,6 +2743,9 @@ public: const enumerators& get_enumerators() const; + const enumerators& + get_sorted_enumerators() const; + enumerators& get_enumerators(); diff --git a/src/abg-comparison.cc b/src/abg-comparison.cc index 1207729c..228909dc 100644 --- a/src/abg-comparison.cc +++ b/src/abg-comparison.cc @@ -9541,7 +9541,8 @@ corpus_diff::priv::ensure_lookup_tables_populated() if (!is_user_defined_type(t)) continue; - string repr = abigail::ir::get_pretty_representation(t, true); + string repr = + abigail::ir::get_pretty_representation(t, /*internal=*/false); deleted_unreachable_types_[repr] = t; } @@ -9563,7 +9564,8 @@ corpus_diff::priv::ensure_lookup_tables_populated() if (!is_user_defined_type(t)) continue; - string repr = abigail::ir::get_pretty_representation(t, true); + string repr = + abigail::ir::get_pretty_representation(t, /*internal=*/false); // Let's see if the inserted type we are looking at was // reported as deleted as well. @@ -9847,7 +9849,7 @@ corpus_diff::priv::added_unreachable_type_is_suppressed(const type_base *t)const if (!t) return false; - string repr = abigail::ir::get_pretty_representation(t, /*internal=*/true); + string repr = abigail::ir::get_pretty_representation(t, /*internal=*/false); string_type_base_sptr_map::const_iterator i = suppressed_added_unreachable_types_.find(repr); if (i == suppressed_added_unreachable_types_.end()) @@ -9869,7 +9871,7 @@ corpus_diff::priv::deleted_unreachable_type_is_suppressed(const type_base *t) co if (!t) return false; - string repr = abigail::ir::get_pretty_representation(t, /*internal=*/true); + string repr = abigail::ir::get_pretty_representation(t, /*internal=*/false); string_type_base_sptr_map::const_iterator i = suppressed_deleted_unreachable_types_.find(repr); if (i == suppressed_deleted_unreachable_types_.end()) diff --git a/src/abg-corpus.cc b/src/abg-corpus.cc index 7cf2d11d..4a4cb8bc 100644 --- a/src/abg-corpus.cc +++ b/src/abg-corpus.cc @@ -774,7 +774,7 @@ corpus::recording_types_reachable_from_public_interface_supported() void corpus::record_type_as_reachable_from_public_interfaces(const type_base& t) { - string repr = get_pretty_representation(&t, /*internal=*/true); + string repr = get_pretty_representation(&t, /*internal=*/false); interned_string s = t.get_environment().intern(repr); priv_->get_public_types_pretty_representations()->insert(s); } @@ -792,7 +792,7 @@ corpus::record_type_as_reachable_from_public_interfaces(const type_base& t) bool corpus::type_is_reachable_from_public_interfaces(const type_base& t) const { - string repr = get_pretty_representation(&t, /*internal=*/true); + string repr = get_pretty_representation(&t, /*internal=*/false); interned_string s = t.get_environment().intern(repr); return (priv_->get_public_types_pretty_representations()->find(s) diff --git a/src/abg-ir.cc b/src/abg-ir.cc index fc3437d6..06dfbe20 100644 --- a/src/abg-ir.cc +++ b/src/abg-ir.cc @@ -8845,6 +8845,8 @@ get_type_name(const type_base* t, bool qualified, bool internal) return fn_type->get_cached_name(internal); } + const environment&env = d->get_environment(); + // All anonymous types of a given kind get to have the same internal // name for internal purpose. This to allow them to be compared // among themselves during type canonicalization. @@ -8863,10 +8865,18 @@ get_type_name(const type_base* t, bool qualified, bool internal) if (qualified) return d->get_qualified_name(internal); - const environment&env = d->get_environment(); return env.intern(get_internal_integral_type_name(t)); } + if (d->get_is_anonymous()) + { + if (is_class_or_union_type(t) || is_enum_type(t)) + return env.intern + (get_class_or_enum_flat_representation (*t, "", + /*one_line=*/true, + internal, qualified)); + } + if (qualified) return d->get_qualified_name(internal); return d->get_name(); @@ -9563,6 +9573,165 @@ get_class_or_union_flat_representation(const class_or_union_sptr& cou, internal, qualified_names);} +/// Get the flat representation of an instance of @ref enum_type_decl +/// type. +/// +/// The flat representation of a given @ref enum_type_decl type is the +/// actual definition of the type, for instance: +/// +/// enum {E_0 =0, E_1 = 1} +/// +///@param enum_type the enum type to consider. +/// +///@param indent the identation spaces to use in the representation. +/// +///@param one_line if true, then the flat representation stands on one +///line. Otherwise, it stands on multiple lines. +/// +///@param qualified_names use qualified names when applicable. +///Typically, if this is true, the name of the enum is going to be +///qualified. +/// +///@return the resulting flat representation. +string +get_enum_flat_representation(const enum_type_decl& enum_type, + const string& indent, bool one_line, + bool qualified_names) +{ + string repr; + std::ostringstream o; + string local_indent = " "; + + repr = indent + "enum "; + + if (!enum_type.get_is_anonymous()) + o << (qualified_names + ? enum_type.get_qualified_name() + : enum_type.get_name()) + " "; + + o << "{"; + + if (!one_line) + o << "\n"; + + for (const auto &enumerator : enum_type.get_sorted_enumerators()) + { + if (!one_line) + o << "\n" + indent; + + o << enumerator.get_name() + "=" << enumerator.get_value() << ", "; + } + + if (!one_line) + o << "\n" + indent << "}"; + else + o << "}"; + + repr =o.str(); + + return repr; +} + +/// Get the flat representation of an instance of @ref enum_type_decl +/// type. +/// +/// The flat representation of a given @ref enum_type_decl type is the +/// actual definition of the type, for instance: +/// +/// enum {E_0 =0, E_1 = 1} +/// +///@param enum_type the enum type to consider. +/// +///@param indent the identation spaces to use in the representation. +/// +///@param one_line if true, then the flat representation stands on one +///line. Otherwise, it stands on multiple lines. +/// +///@param qualified_names use qualified names when applicable. +///Typically, if this is true, the name of the enum is going to be +///qualified. +/// +///@return the resulting flat representation. +string +get_enum_flat_representation(const enum_type_decl* enum_type, + const string& indent, bool one_line, + bool qualified_names) +{ + if (!enum_type) + return ""; + + return get_enum_flat_representation(*enum_type, indent, + one_line, qualified_names); +} + +/// Get the flat representation of an instance of @ref enum_type_decl +/// type. +/// +/// The flat representation of a given @ref enum_type_decl type is the +/// actual definition of the type, for instance: +/// +/// enum {E_0 =0, E_1 = 1} +/// +///@param enum_type the enum type to consider. +/// +///@param indent the identation spaces to use in the representation. +/// +///@param one_line if true, then the flat representation stands on one +///line. Otherwise, it stands on multiple lines. +/// +///@param qualified_names use qualified names when applicable. +///Typically, if this is true, the name of the enum is going to be +///qualified. +/// +///@return the resulting flat representation. +string +get_enum_flat_representation(const enum_type_decl_sptr& enum_type, + const string& indent, bool one_line, + bool qualified_names) +{ + return get_enum_flat_representation(enum_type.get(), + indent, one_line, + qualified_names); +} + +/// Get the flat representation of an instance of @ref enum_type_decl +/// type. +/// +/// The flat representation of a given @ref enum_type_decl type is the +/// actual definition of the type, for instance: +/// +/// enum {E_0 =0, E_1 = 1} +/// +///@param enum_type the enum type to consider. +/// +///@param indent the identation spaces to use in the representation. +/// +///@param one_line if true, then the flat representation stands on one +///line. Otherwise, it stands on multiple lines. +/// +///@param qualified_names use qualified names when applicable. +///Typically, if this is true, the name of the enum is going to be +///qualified. +/// +///@return the resulting flat representation. +string +get_class_or_enum_flat_representation(const type_base& coe, + const string& indent, + bool one_line, + bool internal, + bool qualified_name) + +{ + string repr; + if (const class_or_union* cou = is_class_or_union_type(&coe)) + repr = get_class_or_union_flat_representation(cou, indent, one_line, + internal, qualified_name); + else if (const enum_type_decl* enom = is_enum_type(&coe)) + repr = get_enum_flat_representation(*enom, indent, one_line, qualified_name); + + return repr; +} + /// Get the textual representation of a type for debugging purposes. /// /// If the type is a class/union, this shows the data members, virtual @@ -18578,6 +18747,7 @@ class enum_type_decl::priv { type_base_sptr underlying_type_; enumerators enumerators_; + mutable enumerators sorted_enumerators_; friend class enum_type_decl; @@ -18643,6 +18813,33 @@ enum_type_decl::enumerators& enum_type_decl::get_enumerators() {return priv_->enumerators_;} +/// Get the lexicographically sorted vector of enumerators. +/// +/// @return the lexicographically sorted vector of enumerators. +const enum_type_decl::enumerators& +enum_type_decl::get_sorted_enumerators() const +{ + if (priv_->sorted_enumerators_.empty()) + { + for (auto e = get_enumerators().rbegin(); + e != get_enumerators().rend(); + ++e) + priv_->sorted_enumerators_.push_back(*e); + + std::sort(priv_->sorted_enumerators_.begin(), + priv_->sorted_enumerators_.end(), + [](enum_type_decl::enumerator& l, + enum_type_decl::enumerator& r) + { + if (l.get_name() == r.get_name()) + return l.get_value() < r.get_value(); + return (l.get_name() < r.get_name()); + }); + } + + return priv_->sorted_enumerators_; +} + /// Get the pretty representation of the current instance of @ref /// enum_type_decl. /// @@ -18668,6 +18865,10 @@ enum_type_decl::get_pretty_representation(bool internal, if (internal && get_is_anonymous()) r += get_type_name(this, qualified_name, /*internal=*/true); + else if (get_is_anonymous()) + r += get_enum_flat_representation(*this, "", + /*one_line=*/true, + qualified_name); else r += decl_base::get_pretty_representation(internal, qualified_name); diff --git a/tests/data/test-annotate/libtest23.so.abi b/tests/data/test-annotate/libtest23.so.abi index dcbf8edc..6ca5c79d 100644 --- a/tests/data/test-annotate/libtest23.so.abi +++ b/tests/data/test-annotate/libtest23.so.abi @@ -137,54 +137,54 @@ - - - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + + + @@ -201,7 +201,7 @@ - + @@ -209,11 +209,11 @@ - + - + @@ -235,12 +235,12 @@ - + - + @@ -263,7 +263,7 @@ - + @@ -311,7 +311,7 @@ - + @@ -321,7 +321,7 @@ - + @@ -330,7 +330,7 @@ - + @@ -341,9 +341,9 @@ - + - + @@ -352,13 +352,13 @@ - + - + - + - + @@ -367,13 +367,13 @@ - + - + - + - + @@ -384,11 +384,11 @@ - + - + @@ -399,7 +399,7 @@ - + @@ -412,9 +412,9 @@ - + - + @@ -427,7 +427,7 @@ - + @@ -467,7 +467,7 @@ - + @@ -491,12 +491,12 @@ - + - + - - + + @@ -505,7 +505,7 @@ - + @@ -514,7 +514,7 @@ - + @@ -525,7 +525,7 @@ - + @@ -534,7 +534,7 @@ - + @@ -543,18 +543,18 @@ - + - + - + - + @@ -563,9 +563,9 @@ - + - + @@ -574,7 +574,7 @@ - + @@ -583,18 +583,18 @@ - + - + - + - + @@ -603,7 +603,7 @@ - + @@ -612,16 +612,16 @@ - + - + - + @@ -632,7 +632,7 @@ - + @@ -643,7 +643,7 @@ - + @@ -677,7 +677,7 @@ - + @@ -690,18 +690,18 @@ - + - + - + @@ -712,7 +712,7 @@ - + @@ -721,31 +721,31 @@ - + - + - + - + - + - + @@ -754,7 +754,7 @@ - + @@ -763,7 +763,7 @@ - + @@ -772,7 +772,7 @@ - + @@ -783,7 +783,7 @@ - + @@ -794,7 +794,7 @@ - + @@ -805,7 +805,7 @@ - + @@ -827,7 +827,7 @@ - + @@ -835,7 +835,7 @@ - + @@ -847,7 +847,7 @@ - + @@ -857,7 +857,7 @@ - + @@ -866,7 +866,7 @@ - + @@ -879,7 +879,7 @@ - + @@ -890,7 +890,7 @@ - + @@ -898,7 +898,7 @@ - + @@ -906,9 +906,9 @@ - + - + @@ -918,7 +918,7 @@ - + @@ -927,7 +927,7 @@ - + @@ -936,9 +936,9 @@ - + - + @@ -1002,7 +1002,7 @@ - + @@ -1024,9 +1024,9 @@ - + - + @@ -1080,9 +1080,9 @@ - + - + diff --git a/tests/data/test-annotate/test-anonymous-members-0.o.abi b/tests/data/test-annotate/test-anonymous-members-0.o.abi index e272d213..d75e67ee 100644 --- a/tests/data/test-annotate/test-anonymous-members-0.o.abi +++ b/tests/data/test-annotate/test-anonymous-members-0.o.abi @@ -13,15 +13,15 @@ - - + + - - + + - + @@ -29,11 +29,11 @@ - - + + - - + + @@ -103,7 +103,7 @@ - + @@ -111,7 +111,7 @@ - + diff --git a/tests/data/test-annotate/test15-pr18892.so.abi b/tests/data/test-annotate/test15-pr18892.so.abi index d3616699..b749a0a6 100644 --- a/tests/data/test-annotate/test15-pr18892.so.abi +++ b/tests/data/test-annotate/test15-pr18892.so.abi @@ -10883,7 +10883,7 @@ - + @@ -35469,7 +35469,7 @@ - + @@ -35517,9 +35517,9 @@ - + - + diff --git a/tests/data/test-annotate/test18-pr19037-libvtkRenderingLIC-6.1.so.abi b/tests/data/test-annotate/test18-pr19037-libvtkRenderingLIC-6.1.so.abi index 0a6893b4..adff58f8 100644 --- a/tests/data/test-annotate/test18-pr19037-libvtkRenderingLIC-6.1.so.abi +++ b/tests/data/test-annotate/test18-pr19037-libvtkRenderingLIC-6.1.so.abi @@ -8146,7 +8146,7 @@ - + @@ -12813,7 +12813,7 @@ - + @@ -21260,18 +21260,8 @@ - - - - - - - - - - - - + + @@ -21279,6 +21269,16 @@ + + + + + + + + + + @@ -24118,18 +24118,8 @@ - - - - - - - - - - - - + + @@ -24137,6 +24127,16 @@ + + + + + + + + + + @@ -27487,7 +27487,7 @@ - + diff --git a/tests/data/test-annotate/test19-pr19023-libtcmalloc_and_profiler.so.abi b/tests/data/test-annotate/test19-pr19023-libtcmalloc_and_profiler.so.abi index ea12beac..97490f59 100644 --- a/tests/data/test-annotate/test19-pr19023-libtcmalloc_and_profiler.so.abi +++ b/tests/data/test-annotate/test19-pr19023-libtcmalloc_and_profiler.so.abi @@ -20183,7 +20183,7 @@ - + @@ -25068,21 +25068,21 @@ - + - + - + @@ -32572,7 +32572,7 @@ - + diff --git a/tests/data/test-annotate/test20-pr19025-libvtkParallelCore-6.1.so.abi b/tests/data/test-annotate/test20-pr19025-libvtkParallelCore-6.1.so.abi index 4b4166bf..3de32474 100644 --- a/tests/data/test-annotate/test20-pr19025-libvtkParallelCore-6.1.so.abi +++ b/tests/data/test-annotate/test20-pr19025-libvtkParallelCore-6.1.so.abi @@ -21731,7 +21731,7 @@ - + diff --git a/tests/data/test-diff-dwarf/test43-PR22913-report-0.txt b/tests/data/test-diff-dwarf/test43-PR22913-report-0.txt index fc35f21d..d0515c69 100644 --- a/tests/data/test-diff-dwarf/test43-PR22913-report-0.txt +++ b/tests/data/test-diff-dwarf/test43-PR22913-report-0.txt @@ -4,8 +4,8 @@ Variables changes summary: 0 Removed, 0 Changed, 0 Added variable 1 function with some indirect sub-type change: [C] 'function char f1(Struct1Ptr)' has some indirect sub-type changes: - parameter 1 of type 'typedef Struct1Ptr' has sub-type changes: - underlying type '__anonymous_struct__1*' changed: + parameter 1 of type 'typedef Struct1Ptr' changed: + underlying type 'struct {char m1; char m2;}*' changed: in pointed to type 'struct {char m1; char m2;}': type size changed from 16 to 8 (in bits) 1 data member deletion: diff --git a/tests/data/test-read-dwarf/PR22122-libftdc.so.abi b/tests/data/test-read-dwarf/PR22122-libftdc.so.abi index a1f8778c..501658df 100644 --- a/tests/data/test-read-dwarf/PR22122-libftdc.so.abi +++ b/tests/data/test-read-dwarf/PR22122-libftdc.so.abi @@ -357,136 +357,133 @@ - - - - - + + + + + - - - - + + - - - + + - - - + + - - - - - + + + + - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + - - - + + + + + + - + - - + + - - + + - + - + - + - + - + - + - + - + - + - + @@ -494,105 +491,105 @@ - + - + - + - + - + - + - + - + - - + + - + - + - + - + - + - + - + - + @@ -600,91 +597,91 @@ - + - + - + - + - + - + - + - + - + - - - - + + + + - + - + - + - + - + - - - + + + - + @@ -692,96 +689,96 @@ - + - + - - + + - + - + - + - - + + - + - + - + - + - + - + - - + + - + - + - + - - - + + + - - - - + + + + - - - + + + - + - + - - - + + + - + @@ -790,7 +787,7 @@ - + @@ -800,90 +797,90 @@ - + - - + + - + - + - + - + - + - + - + - + - + - + - - - - + + + + - + - - - - + + + + - + - - - - + + + + - + - - - - + + + + @@ -891,13 +888,13 @@ - + - + @@ -915,26 +912,26 @@ - + - + - - + + - + @@ -946,19 +943,19 @@ - + - + - + @@ -970,90 +967,90 @@ - - + + - - + + - + - + - + - - + + - + - - + + - - - + + + - - + + - + - - + + - - + + @@ -1061,8 +1058,8 @@ - - + + @@ -1070,8 +1067,8 @@ - - + + @@ -1079,29 +1076,29 @@ - + - - + + - - + + - - + + @@ -1109,14 +1106,14 @@ - + - - + + @@ -1124,8 +1121,8 @@ - - + + @@ -1133,8 +1130,8 @@ - - + + @@ -1142,7 +1139,7 @@ - + @@ -1150,246 +1147,246 @@ - + - - + + - - + + - + - - + + - + - + - + - - + + - + - + - - + + - + - + - - + + - - - + + + - - + + - - + + - - + + - - + + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - - + + - - + + - + - + - - + + - + - + - + - + - + - + - + @@ -1528,7 +1525,7 @@ - + @@ -1653,14 +1650,14 @@ - + - + @@ -1678,7 +1675,7 @@ - + @@ -1692,7 +1689,7 @@ - + @@ -1704,7 +1701,7 @@ - + @@ -1762,28 +1759,28 @@ - + - + - + - + - + - - + + - + @@ -1795,15 +1792,15 @@ - + - + - + @@ -1814,11 +1811,11 @@ - + - + @@ -1876,34 +1873,34 @@ - - + + - + - - + + - + - + - + - - + + - + - + @@ -2310,7 +2307,7 @@ - + @@ -2956,12 +2953,12 @@ - - + + - + @@ -2969,133 +2966,133 @@ - + - + - + - + - + - + - + - + - + - + - - - + + + - - - - + + + + - + - + - + - + - - + + - + - - + + - + - - - + + + - - + + - + - + - - + + - + - + - - + + - + - + - + @@ -3103,499 +3100,489 @@ - - + + - + - + - + - + - + - + - + - - + + - + - + - + - + - + - + - + - + - + - + - + - + - - + + - - - + + + - - - + + + - + - + - + - + - + - + - + - + - + - + - - - + + + - - - + + + - - - + + + - - + + - + - + - + - + - + - + - + - + - + - + - - + + - + - + - + - - + + - - + + - + - + - + - - + + - + - + - - + + - - - + + + - + - + - - + + - - - + + + - - - + + + - + - + - - - + + + - - + + - + - - - - - + + + + + + + - - - - - + + + + + - + - - - - + + + + - - + + - - - - - - - - + + + + - - - - - - - - - - - - + + + + + + + + + - + - - - - - - - - - - + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - + + + + + + - - + + - + - + - + - + - - - + + + - - + + - + - - + + - - + + - + - + @@ -3603,429 +3590,429 @@ - - - + + + - + - - - - + + + + - - - - + + + + - + - - + + - + - + - + - + - + - - + + - - - + + + - + - - + + - + - - + + - + - - - - + + + + - + - + - + - + - - + + - - + + - - + + - + - + - + - + - - + + - + - + - + - + - - - + + + - - - + + + - + - - - + + + - - - + + + - + - - - - + + + + - - - - + + + + - + - - - + + + - + - - - + + + - - - - + + + + - - - - + + + + - - - + + + - + - - - + + + - + - - - + + + - + - + - + - + - - + + - + - + - - - - + + + + - + - - - + + + - + - + - + - - - - + + + + - + - + - - + + - + - + - - - + + + - + - - - - + + + + - + - - - + + + - + - - + + - - - + + + - - + + - + - + - - - - + + + + - + - - + + - - + + - + - + - - + + - + - + - + - + @@ -4035,384 +4022,384 @@ - + - - + + - - + + - - + + - + - - + + - - + + - + - + - + - - + + - + - + - + - - + + - - + + - + - + - + - + - + - + - + - + - + - + - - + + - + - + - - + + - + - + - + - + - + - + - - - + + + - - - + + + - + - - + + - - + + - + - + - - - + + + - - - + + + - - + + - + - - + + - - + + - + - + - - + + - + - - + + - + - + - - + + - + - + - + - + - + - + - + - + - + - + - - - + + + - + - - + + - + - + - + - + - - + + - - - + + + - + - + - + - + - + - + - - - + + + - + - + - - + + - + - + - + @@ -4420,566 +4407,535 @@ - - + + - + - + - + - + - + - + - + - - + + - + - - + + - + - + - + - - + + - + - + - + - - - + + + - - - + + + - - - + + + - - - + + + - - + + - + - + - + - - + + - + - + - - + + - + - - + + - - + + - + - + - - + + - + - - + + - + - - + + - + - - + + - - + + - + - - + + - - + + - + - - + + - - - + + + - + - - - + + + - - + + - - - + + + - - - + + + - - + + - + - - - + + + - - - + + + - + - - + + - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - + + - - - - - - + + + + - - - + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - + + + - - - - - + + + + - + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + - - + + - - - - - + + + + + - - - + + + - + - + - + - + - - + + - + - - + + - - + + - - - + + + - - - + + + - + - + - + - + - + - + - - - + + + - - - + + + - + - + - - + + - + @@ -4987,963 +4943,963 @@ - + - + - - + + - - - - + + + + - - - + + + - + - - - - + + + + - - - - + + + + - - - + + + - - - + + + - + - + - + - + - - - + + + - + - - - + + + - + - - - + + + - - - + + + - + - - - + + + - - - + + + - + - - - - + + + + - - - + + + - + - - - + + + - + - + - + - + - - - - + + + + - + - - - - + + + + - + - + - + - - - + + + - + - + - - + + - + - + - + - + - + - + - + - - + + - + - + - - - + + + - - - + + + - + - - - + + + - - - + + + - + - - - - + + + + - - - + + + - + - - - - - + + + + + - - - + + + - + - - + + - + - + - + - - + + - - + + - + - + - + - + - + - + - + - + - + - - - + + + - - - + + + - - - + + + - - + + - + - + - - + + - + - - - + + + - - + + - + - + - + - + - - + + - + - + - + - + - + - + - + - + - + - + - - + + - + - + - + - + - + - + - + - + - - + + - + - + - + - + - + - + - + - - + + - + - - + + - - + + - + - + - - - + + + - - - + + + - + - + - + - + - - + + - - + + - - + + - + - + - - + + - - - + + + - - - + + + - - - + + + - + - + - - + + - - - + + + - + - + - - + + - + - + - - - - - + + + + + - + - + - + - + - - - + + + - - - + + + - - - + + + - - + + - - - + + + - + - - - + + + - - - + + + - + - + - - + + - - - + + + - - - + + + - + - - - + + + - - - - + + + + - - - - + + + + - - + + - - + + - - + + - + - - + + - + - - + + - + - + - + - + - + - + - - + + - + - - - + + + - + - - + + - + - + - + - + - + - + - - + + - + - + - - + + - + - + - - + + - + - + - - + + @@ -5952,1659 +5908,1629 @@ - - + + - + - - + + - + - - + + - + - - + + - + - - - + + + - - - + + + - - - + + + - + - - - + + + - + - + - - - + + + - - + + - + - + - - + + - - + + - + - + - + - - + + - - + + - - + + - - - + + + - + - - - + + + - - + + - - - + + + - + - - + + - + - - + + - - - + + + - + - + - - + + - - + + - + - + - - + + - - + + - - - + + + - + - + - + - - - + + + - - - + + + - + - + - - - + + + - - - + + + - - + + - + - + - + - - + + - - + + - - - + + + - - + + - - - + + + - - - + + + - + - + - - + + - - + + - - - + + + - + - + - + - + - + - + - + - + - + - + - + - + - - + + - - + + - - + + - - + + - - + + - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - + + + + + + - - - - - - - - + + + + + - - - + + - + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + - + - + - + - + - - - - + + + + - - + + - - + + - + - - + + - + - + - - + + - + - - - + + + - - + + - + - - + + - + - - - + + + - - - + + + - - - + + + - - - + + + - + - - + + - - + + - - - - - - + + + + + + - - - + + + - + - - - - + + + + - + - + - + - + - - + + - + - - + + - + - - - - + + + + - - - + + + - - - - + + + + - - + + - + - - - + + + - - - + + + - + - - + + - - + + - + - - + + - - + + - + - - + + - - + + - + - - - - + + + + - - - - + + + + - + - + - - + + - - - + + + - - + + - - + + - + - + - + - + - - + + - - + + - - - + + + - - - + + + - - - + + + - - + + - - + + - + - + - - - + + + - - - + + + - + - + - - - + + + - - + + - + - - - - - + + + + + - - + + - + - - - - - + + + + + - + - - - - + + + + - - + + - - + + - - + + - - + + - - - + + + - + - - + + - - - + + + - - + + - + - - + + - - - + + + - + - + - + - - - + + + - - + + - + - + - - + + - - - + + + - - + + - + - + - - + + - - - + + + - + - - - + + + - - - + + + - + - - + + - + - - + + - + - - - + + + - + - - - + + + - + - - - + + + - - - + + + - + - - - + + + - - - - + + + + - - - - + + + + - - - + + + - + - + - - - + + + - - - + + + - - - + + + - - + + - - + + - + - + - + - + - - + + - + - + - + - - - - + + + + - - - + + + - + - - + + - + - - - + + + - + - + - + - - + + - + - - + + - + - + - + - - - + + + - + - - - - + + + + - + - - - + + + - + - + - - + + - - + + - - + + - + - + - - + + - + - + - + - + - - + + - - + + - + - + - - - - + + + + - - - + + + - + - + - - - + + + - - - + + + - - - + + + - + - + - - + + - + - + - - - + + + - - - + + + - - - + + + - + - + - + - + - - - + + + - - - + + + - - - + + + - - + + - - + + - - + + - - + + - + - + - - + + - + - - - + + + - + - - + + - - + + - - + + - - + + - + - + - - + + - + - + - + @@ -7612,724 +7538,713 @@ - - + + - + - + - + - + - - - + + + - + - + - + - - + + - - + + - - - + + + - + - + - - + + - + - + - + - + - + - + - + - + - + - + - + - + - - + + - - - + + + - - - + + + - + - + - + - - - + + + - - - + + + - - - + + + - - - + + + - - + + - + - + - + - - - + + + - + - + - + - - - + + + - - - + + + - - - + + + - + - + - + - - + + - + - - + + - - + + - - + + - + - + - - - + + + - - - + + + - + - - - + + + - + - - - + + + - - - + + + - + - + - - + + - - - + + + - - - + + + - - + + - + - - - + + + - + - - + + - + - + - - - + + + - - + + - - + + - + - + - - - + + + - + - + - + - + - - - + + + - - - + + + - - + + - - + + - - - + + + - + - + - + - + - - + + - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + - - - + + + - + - + - - + + - + - + - - + + - + - + - + - - - + + + - + - + - - + + - + - + - + - + - - + + - - + + - + - + - - + + - - - + + + - + - - - + + + - - + + - - + + - - - + + + - - - + + + - - + + - + - + - - - + + + - + - + - + @@ -8337,1252 +8252,1298 @@ - - - + + + - - - + + + - + - - - - + + + + - - - - + + + + - + - + - + - + - - - - + + + + - - - - + + + + - + - - - + + + - + - - + + - + - + - + - - + + - + - - + + - + - + - - + + - + - + - + - + - - + + - - + + - + - + - + - + - + - + - + - - + + - - - + + + - + - + - - + + - + - + - + - + - + - - + + - - - + + + - - + + - - + + - + - + - + - + - - - + + + - + - - - + + + - + - + - + - - + + - + - - + + - + - + - + - - + + - - + + - + - + - + - - - - + + + + - + - - + + - - - - + + + + - - + + - + - + - + - + - + - - - + + + - - - + + + - - + + - + - + - + - + - + - + - + - - - + + + - - - + + + - + - - - + + + - - + + - - - + + + - - + + - - - + + + - - - + + + - - + + - + - + - + - - + + - - - + + + - - - + + + - - + + - - + + - - + + - + - + - - + + - - - + + + - + - - + + - - - + + + - - + + - - - + + + - - + + - + - + - - + + - + - - - + + + - + - - + + - + - + - + - - + + - - - + + + - - - + + + - - - + + + - + - - - + + + - - + + - + - - - + + + - + - + - - + + - - + + - + - - + + - - - + + + - - - + + + - + - - + + - - - + + + - + - - + + - + - + - - - + + + - + - - + + - + - + - - - + + + - + - + - + - - - + + + - + - + - + - - - + + + - + - - + + - + - + - - + + - + - - - + + + - + - - - + + + - + - - - + + + - + - + - + - - - + + + - + - - - + + + - - + + - + - - + + - + - + - - + + - - - + + + - + - + - - + + - - + + - + - - + + - - - + + + - - - + + + - - + + - - - + + + - + - + - + - - + + - - + + - - + + - - - + + + - + - + - - + + - - + + - + - + - + - + - - + + - - - + + + - - - + + + - + - + - - + + - - + + - - - + + + - + - + - + - - + + - - + + - - - + + + - - + + - - - + + + - - - + + + - + - - + + - - + + - - - + + + - + - + - + - - + + - - + + - - + + - - + + - - - + + + - + - - + + - + - + - + - + - + - + - + - + - + - + - - + + - - + + - - + + - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - + + + + + + + + @@ -9592,278 +9553,212 @@ - - - + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - + + - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + - + - - - - + + + - - - + + - - - + + - - - - - + + + - - - - - - - + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + - + - + - - + + - + - - + + - - + + - - + + - - - - - - - - + + + + + + + + - + - + - + - + - - + + - - + + - - - + + + - - + + - - + + - - + + - + - + @@ -9871,170 +9766,170 @@ - - - + + + - + - - + + - - - + + + - + - - - - + + + + - + - + - - + + - + - - + + - + - - + + - - + + - + - - + + - + - - + + - - + + - + - - + + - - + + - - + + - - + + - + - + - - + + - - + + - - - + + + - - + + - + - - + + - + - + @@ -10042,861 +9937,861 @@ - - - + + + - + - - - - - + + + + + - - - + + + - + - + - + - + - - - + + + - + - - - - + + + + - - - + + + - + - + - - + + - + - - + + - - + + - + - - + + - + - + - + - + - + - - + + - + - - - - + + + + - + - + - + - + - - + + - + - - + + - - + + - + - - + + - + - + - + - + - + - - + + - + - - - - + + + + - + - + - + - - - + + + - + - - - + + + - - + + - - - - + + + + - + - + - + - + - - - - - + + + + + - - - + + + - + - - + + - + - - + + - + - - - + + + - - - + + + - + - - - + + + - - - + + + - - + + - - - + + + - - - + + + - + - - + + - + - - + + - + - + - - + + - - + + - - + + - - + + - + - + - + - - + + - - + + - + - + - + - + - - + + - + - + - + - - + + - - + + - - + + - + - - + + - + - + - - - + + + - - + + - + - + - + - + - - + + - + - + - + - - + + - - + + - - + + - - + + - + - + - + - - + + - + - + - - - - + + + + - - - - + + + + - + - - - - + + + + - + - - - - + + + + - - - + + + - - - - + + + + - - - + + + - + - - - + + + - + - - - + + + - - - + + + - - - + + + - - - - - - + + + + + + - - - + + + - - - - + + + + - - + + - - + + - + - - - + + + - - - - + + + + - + - - + + - + - + - - - + + + - - - + + + - - + + - - - + + + - - + + - + - - + + - + - + - - + + - + - + - + - - + + - + - - - - + + + + - + - - + + - + - + - - + + - + - - - - + + + + - + - - + + - + - + - - + + - + - + - - + + - + - + - + - + - + - + - + - + - + - + - + - - - + + + - + - - - + + + - + - - - + + + @@ -10905,424 +10800,424 @@ - + - - - + + + - - - + + + - + - + - - + + - - - - - + + + + + - - + + - - + + - - - - - + + + + + - - + + - - + + - - - - + + + + - - - - - + + + + + - - - + + + - - - + + + - - - - - + + + + + - + - + - + - - - + + + - - + + - - + + - + - + - + - - - - + + + + - - - + + + - - - + + + - + - + - + - - + + - - + + - + - - - + + + - + - - + + - + - - - + + + - + - + - + - - + + - - + + - - + + - + - + - - + + - - - + + + - - + + - + - - - + + + - + - - - + + + - + - + - + - - + + - - + + - + - - + + - - - + + + - - - + + + - - + + - + - - - + + + - + - - - + + + - - - + + + - - - + + + - - - + + + - - + + - + - + - - + + - + - + - + @@ -11330,1179 +11225,1174 @@ - - + + - + - + - + - + - + - + - + - - + + - + - + - + - + - + - + - + - + - + - + - + - + - - + + - - - + + + - - - + + + - + - + - + - + - - + + - - + + - - + + - + - + - - - + + + - + - + - - - + + + - + - - - + + + - + - - + + - - - + + + - - - + + + - - + + - + - - - + + + - + - - + + - + - + - - + + - - - + + + - + - + - + - - + + - - + + - - + + - - - + + + - + - + - - + + - - - + + + - + - - + + - - + + - - - + + + - + - + - + - - - + + + - - + + - - - + + + - - - + + + - - - + + + - - - + + + - - - + + + - - - + + + - + - - + + - - + + - - + + - - + + - - - + + + - + - - + + - - - + + + - + - - + + - - - + + + - + - - + + - - + + - - - + + + - + - - - + + + - - + + - - + + - + - + - - + + - - + + - + - - - + + + - + - + - - + + - - - + + + - - - + + + - + - + - + - + - - + + - - + + - - - + + + - - + + - - + + - - + + - - - + + + - - - + + + - + - - + + - + - + - + - - - + + + - - - - + + + + - - - + + + - - - + + + - - + + - - + + - + - + - + - + - + - + - + - + - + - - + + - - + + - - + + - + - + - - + + - - + + - - - + + + - - - + + + - + - - + + - - + + - - + + - - + + - - + + - + - - - + + + - - + + - - - + + + - - + + - + - + - + - + - + - + - + - - - - + + + + - - + + - - + + - - + + - - + + - - + + - - + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - + + - - - + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + - + - + - + - - + + - + - + - - + + - - + + - + - + - + - + - + - + - + - - - - + + + + - + - + - + - - + + - + - - + + - + - + - - - + + + - - + + - - - + + + - + - + - + - + - + - - + + - + - - - - - + + + + + - - + + - + - - + + - + - + - + - + - - - + + + - + @@ -12510,183 +12400,183 @@ - + - + - + - + - + - + - + - + - - + + - + - + - - - + + + - + - - - - + + + + - - - - + + + + - + - - - - + + + + - - - - + + + + - + - - - + + + - + - - - + + + - + - - - - + + + + - + - - - - - + + + + + - + - - - + + + - + - - + + - - + + - + - + - + - + - - - + + + - + - + - - - + + + - + @@ -12695,7 +12585,7 @@ - + @@ -12705,376 +12595,376 @@ - + - - + + - + - - - - + + + + - - - - + + + + - - - + + + - - - + + + - + - - - + + + - - - + + + - + - + - - + + - - + + - + - + - + - + - + - + - + - - + + - + - + - + - + - - + + - - + + - + - + - + - - + + - + - - - + + + - - + + - + - - + + - + - + - - + + - + - + - - + + - + - + - - + + - + - - + + - + - + - + - + - + - + - + - - - + + + - - - + + + - - - + + + - - - + + + - - - + + + - + - + - - - + + + - + - + - + - + - - - + + + - - - + + + - - - + + + - - - + + + - - + + - + - + - + - + - + - + - + - - + + - + - + - + @@ -13082,567 +12972,567 @@ - - - + + + - - + + - + - + - + - + - + - + - + - - - + + + - - + + - + - + - - + + - + - + - - + + - - + + - + - - + + - + - - + + - + - + - + - + - + - + - + - + - + - + - + - + - - + + - - - + + + - - - + + + - + - + - + - + - + - + - + - + - + - + - + - - - + + + - + - + - + - + - + - + - - + + - + - + - + - + - + - - + + - + - - + + - + - + - + - + - + - - + + - - - + + + - - - + + + - + - - + + - + - - - + + + - - - + + + - - - + + + - - - + + + - - + + - - - + + + - - + + - - + + - - + + - - + + - - - + + + - + - + - - + + - + - - - + + + - + - - + + - - - + + + - - + + - - + + - + - + - + - - - + + + - + - + - - + + - - - - + + + + - - + + - + - - - + + + - - - + + + - + - + - - + + - + - + - - + + - + - + - + - - - + + + - + - + - + - - - + + + @@ -13651,64 +13541,64 @@ - - + + - + - - + + - + - + - + - - - + + + - + - - - + + + - - + + - - + + - + - + @@ -13716,733 +13606,728 @@ - + - - + + - - - + + + - - - + + + - - + + - - - + + + - + - - - + + + - + - - + + - + - + - - - + + + - - + + - + - + - + - - + + - - + + - - + + - - - + + + - + - + - - + + - - + + - + - + - - + + - - + + - + - + - - + + - - + + - + - + - + - - + + - - + + - - + + - - - + + + - + - + - - + + - - - + + + - + - - - + + + - - - + + + - - + + - + - - + + - + - + - + - - + + - - + + - - - + + + - - + + - - - + + + - - - + + + - + - + - + - - + + - - + + - + - - - + + + - - + + - - - + + + - - - + + + - + - + - - + + - - + + - - - + + + - + - + - + - - + + - - + + - + - - - + + + - + - + - + - - + + - - + + - - - + + + - - + + - - + + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - - + + - - + + - - + + - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + - - - - + + + + - - + + - + - - + + - + - - - - + + + + - - - + + + - - - - + + + + - - - + + + - + - + - + - + - - + + - - + + - + - + - + - - + + - + - + - - + + - - + + - + - + - + - - + + - + - + - + - - - + + + - + - - - + + + @@ -14450,370 +14335,370 @@ - + - - + + - + - + - - + + - + - + - - + + - + - + - + - + - + - - - + + + - - - + + + - + - + - - + + - - + + - - + + - + - + - - - + + + - - + + - - - + + + - + - - - + + + - - - + + + - - - + + + - - - + + + - - + + - + - + - + - + - - + + - + - - + + - + - - + + - - - + + + - - - + + + - + - + - + - + - + - - - + + + - + - + - + - - + + - + - + - + - - + + - + - - + + - + - + - - + + - + - - + + - - + + - - - + + + - - - + + + - - - + + + - - - - + + + + - - - - + + + + - - - - + + + + - - + + - + - + - - + + - + - + - + @@ -14821,145 +14706,145 @@ - - + + - + - + - + - + - + - + - + - - + + - + - + - + - + - + - + - + - + - + - + - + - - + + - + - + - + - - + + - + - + - + - + - - + + - + - + - - + + @@ -14968,498 +14853,489 @@ - - + + - + - - + + - + - + - - - + + + - + - + - - - + + + - - - + + + - + - - + + - - + + - - - + + + - - - + + + - + - + - - + + - - + + - + - - + + - - + + - + - + - - + + - - + + - + - + - - - + + + - - + + - - - + + + - + - + - - - + + + - - - + + + - - - + + + - + - + - - + + - - + + - - - + + + - + - + - - + + - - + + - - - + + + - + - + - - + + - - + + - - - + + + - + - + - + - + - + - + - + - - + + - - + + - - + + - + - - + + - - + + - - + + - - + + - - + + - - + + - - + + - - + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + - - + + - + - - - + + + - - - + + + - - - + + + - + - + - + - + @@ -15467,222 +15343,222 @@ - - - + + + - - - + + + - - - + + + - - + + - + - - + + - + - - + + - + - - + + - + - + - + - + - + - + - - + + - + - + - - - - + + + + - + - - - - + + + + - + - + - - - + + + - - + + - + - + - + - + - - - + + + - - - + + + - - - + + + - + - + - + - + - - - - + + + + - - - - + + + + - - - + + + - - - + + + - + - - - + + + - + - - - + + + @@ -15690,626 +15566,626 @@ - + - - + + - + - + - - - - + + + + - - - - + + + + - - - - + + + + - + - - + + - - + + - + - + - + - + - + - - - + + + - - - + + + - - - + + + - - - + + + - - - + + + - - - + + + - - - + + + - + - + - + - + - + - - + + - + - + - + - + - - + + - + - - + + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - - + + - - + + - + - - + + - + - + - + - + - - + + - - + + - + - + - + - - + + - + - + - + - + - - - + + + - + - + - - - + + + - + - - + + - + - - + + - + - + - + - - + + - + - + - + - - + + - + - + - + - - + + - + - + - + - + - + - + - - + + - + - + - - + + - + - + - + - + - - + + - + - - + + - - + + - - + + - - + + - + - + - + - + - - + + - + - - - + + + - - + + - + - + - - + + - - - + + + - + - + - + - + - + - - - + + + - + - + - + - - + + - + - + @@ -16317,657 +16193,657 @@ - - + + - + - + - + - + - + - + - + - - + + - + - - - + + + - - - + + + - - - + + + - - - + + + - - + + - + - + - - + + - - + + - + - + - - + + - - - + + + - - - + + + - - + + - - + + - + - + - - - - + + + + - - - - + + + + - + - - + + - - + + - - + + - - + + - + - - + + - + - - + + - - - + + + - - + + - + - - + + - + - - - + + + - - - + + + - - - + + + - - - + + + - + - - + + - + - + - - + + - + - - + + - + - + - + - + - + - - + + - + - - + + - + - - + + - + - + - - - + + + - - - + + + - + - - + + - - - + + + - - - + + + - + - + - + - + - - - + + + - - + + - + - + - + - + - - + + - + - + - + - - + + - - + + - + - + - - + + - + - - + + - + - - - + + + - - - + + + - + - - - + + + - - - + + + - - - - + + + + - - + + - + - - - + + + - - - - + + + + - - - + + + - + - - + + - - + + - + - + - - + + - + - - + + - + - + - - + + - + - - + + - + - + - - + + - + - - + + - + - + - - + + - + - + - - + + - + - + - - + + - + - + - - + + - + - + - - + + @@ -16976,667 +16852,667 @@ - - + + - + - - + + - + - - + + - + - - + + - + - - + + - + - - + + - + - - + + - + - - + + - + - - + + - - + + - + - - + + - + - - + + - + - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + - - - - + + + + - - - + + + - - - + + + - - - + + + - - + + - - + + - - + + - - - + + + - - + + - - + + - - - + + + - - + + - + - - + + - - + + - - + + - + - + - - - + + + - + - + - + - - - + + + - + - + - - + + - - + + - - + + - - - + + + - + - - + + - - + + - - - + + + - - - + + + - - - + + + - - - + + + - + - - - + + + - + - - + + - + - + - - + + - - - + + + - + - + - - + + - - - + + + - + - + - - + + - - - + + + - - + + - - + + - + - - + + - + - - - + + + - - - + + + - - - + + + - + - + - - - + + + - - - + + + - + - - + + - - - + + + - - + + - - + + - - - + + + - - - + + + - + - + - - + + - - - + + + - - - + + + - + - + - - + + - - - + + + - - - + + + - + - + - - + + - - - + + + - - - + + + - + - + - - + + - + - + - + - + - + - + - + - + - - + + - - - - - - - - - - - - + + + + + + + + + + + + - + - - + + - + - + - + - + - - - + + + - + - - - + + + - + - - + + diff --git a/tests/data/test-read-dwarf/libtest23.so.abi b/tests/data/test-read-dwarf/libtest23.so.abi index 459527f0..53fa11ad 100644 --- a/tests/data/test-read-dwarf/libtest23.so.abi +++ b/tests/data/test-read-dwarf/libtest23.so.abi @@ -92,30 +92,30 @@ - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + - - - - - + + + + + @@ -125,16 +125,16 @@ - + - + - + @@ -148,10 +148,10 @@ - + - + @@ -168,7 +168,7 @@ - + @@ -204,62 +204,62 @@ - + - + - + - - + + - - - - + + + + - - - - + + + + - + - + - + @@ -267,8 +267,8 @@ - - + + @@ -276,7 +276,7 @@ - + @@ -302,7 +302,7 @@ - + @@ -315,91 +315,91 @@ - + - + - + - + - + - + - - + + - - + + - - + + - + - - + + - - + + - + - + - - + + - + - + @@ -418,77 +418,77 @@ - + - + - + - + - + - + - + - - + + - + - + - + - + - + - + - + - + @@ -500,71 +500,71 @@ - + - + - + - + - + - + - + - + - - + + - + - + - - + + @@ -600,7 +600,7 @@ - + @@ -612,8 +612,8 @@ - - + + @@ -644,8 +644,8 @@ - - + + diff --git a/tests/data/test-read-dwarf/test-libandroid.so.abi b/tests/data/test-read-dwarf/test-libandroid.so.abi index 29eda64b..32c1a0a4 100644 --- a/tests/data/test-read-dwarf/test-libandroid.so.abi +++ b/tests/data/test-read-dwarf/test-libandroid.so.abi @@ -8037,17 +8037,7 @@ - - - - - - - - - - - + @@ -8067,6 +8057,16 @@ + + + + + + + + + + @@ -18980,7 +18980,6 @@ - @@ -19098,6 +19097,7 @@ + @@ -24528,6 +24528,8 @@ + + @@ -25770,7 +25772,7 @@ - + @@ -30500,7 +30502,7 @@ - + @@ -31186,7 +31188,7 @@ - + @@ -35836,7 +35838,7 @@ - + @@ -37487,7 +37489,7 @@ - + @@ -37510,7 +37512,7 @@ - + @@ -37590,7 +37592,7 @@ - + @@ -37613,7 +37615,7 @@ - + @@ -37693,7 +37695,7 @@ - + diff --git a/tests/data/test-read-dwarf/test16-pr18904.so.abi b/tests/data/test-read-dwarf/test16-pr18904.so.abi index 805ff811..a7cc8997 100644 --- a/tests/data/test-read-dwarf/test16-pr18904.so.abi +++ b/tests/data/test-read-dwarf/test16-pr18904.so.abi @@ -2168,56 +2168,50 @@ - - + + - - - - - - - + - + - + - - + + - - + + - - + + - - + + - - + + - + @@ -2225,14 +2219,14 @@ - + - + @@ -2241,26 +2235,26 @@ - - + + - + - + - + @@ -2268,7 +2262,7 @@ - + @@ -2276,35 +2270,35 @@ - - + + - + - + - + - + - + - + @@ -2319,35 +2313,35 @@ - + - + - - + + - - + + - - + + - + @@ -2355,7 +2349,7 @@ - + @@ -2363,85 +2357,85 @@ - - + + - - + + - - + + - - + + - + - + - + - + - + - + - - + + - + - + - + - - + + - - + + - + - + @@ -2450,7 +2444,7 @@ - + @@ -2598,12 +2592,12 @@ - + - + - + @@ -2611,33 +2605,33 @@ - + - + - + - + - + - + - + - + @@ -2645,22 +2639,22 @@ - + - + - + - + - + - + @@ -2668,30 +2662,30 @@ - + - + - + - + - + - + - + @@ -2704,10 +2698,10 @@ - + - + @@ -2716,37 +2710,37 @@ - + - + - + - + - + - + - + - + - + - + @@ -2755,22 +2749,22 @@ - + - + - + - + - + - + @@ -2780,47 +2774,47 @@ - - + + - + - - + + - - - + + + - - + + - - + + - - - + + + @@ -2833,13 +2827,13 @@ - + - + @@ -2847,41 +2841,41 @@ - + - + - + - + - + - - + + - + @@ -2889,151 +2883,151 @@ - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - - + + - + - + - + - + - - + + - - + + - + - + - + - + - + - + - + @@ -3054,29 +3048,29 @@ - + - + - + - + - + - + - + - + @@ -3085,33 +3079,33 @@ - + - + - + - + - + - + @@ -3120,13 +3114,13 @@ - + - + - + @@ -3135,31 +3129,31 @@ - + - + - + - + - + @@ -3168,53 +3162,53 @@ - + - + - + - + - + - + - + - + - + - + - + - + - + - + @@ -3223,40 +3217,40 @@ - + - + - + - + - + - + - + - + - + - + @@ -3277,10 +3271,10 @@ - + - + @@ -3304,81 +3298,81 @@ - + - - + + - - + + - + - + - + - + - + - + - + - + - - - + + + - + - + - + @@ -3387,11 +3381,11 @@ - + - + - + @@ -3399,20 +3393,20 @@ - + - + - + @@ -3420,12 +3414,12 @@ - + - - + + - + @@ -3433,69 +3427,69 @@ - + - + - + - + - - + + - + - + - - + + - + - + - + - + @@ -3503,22 +3497,22 @@ - + - + - - - + + + @@ -3527,44 +3521,44 @@ - + - + - - + + - - + + - - + + - - + + - - + + @@ -3572,12 +3566,12 @@ - - + + - - - + + + @@ -3587,36 +3581,36 @@ - + - + - - + + - - + + - + - + - + @@ -3632,7 +3626,7 @@ - + @@ -3647,21 +3641,21 @@ - + - + - + - + - + @@ -3672,14 +3666,14 @@ - + - + - + @@ -3689,7 +3683,7 @@ - + @@ -3698,18 +3692,18 @@ - + - + - + - + @@ -3718,17 +3712,17 @@ - + - + - + - + @@ -3736,38 +3730,38 @@ - + - - + + - + - - + + - + - + @@ -3776,32 +3770,32 @@ - + - - + + - - + + - + - + @@ -3809,14 +3803,14 @@ - + - + @@ -3834,7 +3828,7 @@ - + @@ -3845,7 +3839,7 @@ - + @@ -3855,21 +3849,21 @@ - + - + - + - + - + - + @@ -3879,7 +3873,7 @@ - + @@ -3887,9 +3881,9 @@ - + - + @@ -3899,7 +3893,7 @@ - + @@ -3907,20 +3901,20 @@ - + - + - + - + @@ -3935,19 +3929,19 @@ - + - + - + - + - + @@ -3956,7 +3950,7 @@ - + @@ -3964,34 +3958,40 @@ - + - + - + - + + + + + + + - - - - - - - - - + + + + + + + + + - - - + + + @@ -4009,7 +4009,7 @@ - + @@ -4081,37 +4081,37 @@ - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + - - - - - - - - - + + + + + + + + + @@ -4144,29 +4144,29 @@ - - - - - - + + + + + + - - - - - - - - + + + + + + + + - - - - - + + + + + @@ -4250,7 +4250,7 @@ - + @@ -4422,8 +4422,8 @@ - - + + @@ -4465,7 +4465,7 @@ - + @@ -4486,12 +4486,12 @@ - + - + @@ -4515,7 +4515,7 @@ - + @@ -4609,8 +4609,8 @@ - - + + @@ -4646,17 +4646,17 @@ - - + + - - + + - - + + @@ -4676,11 +4676,11 @@ - - + + - - + + @@ -4692,14 +4692,14 @@ - - + + - - + + @@ -4796,10 +4796,10 @@ - + - + @@ -4960,7 +4960,7 @@ - + @@ -4979,7 +4979,7 @@ - + @@ -4999,7 +4999,7 @@ - + @@ -5023,7 +5023,7 @@ - + @@ -5037,7 +5037,7 @@ - + @@ -5050,7 +5050,7 @@ - + @@ -5070,7 +5070,7 @@ - + @@ -5171,10 +5171,10 @@ - + - + @@ -5252,7 +5252,7 @@ - + @@ -5287,7 +5287,7 @@ - + @@ -5349,7 +5349,7 @@ - + @@ -5377,7 +5377,7 @@ - + @@ -5392,7 +5392,7 @@ - + @@ -5418,7 +5418,7 @@ - + @@ -5431,7 +5431,7 @@ - + @@ -6274,7 +6274,7 @@ - + @@ -6287,7 +6287,7 @@ - + @@ -6300,7 +6300,7 @@ - + @@ -6323,7 +6323,7 @@ - + @@ -6691,7 +6691,7 @@ - + @@ -6699,13 +6699,13 @@ - + - + @@ -6713,7 +6713,7 @@ - + @@ -6742,7 +6742,7 @@ - + @@ -6750,7 +6750,7 @@ - + @@ -6758,28 +6758,28 @@ - + - + - + - + @@ -6805,10 +6805,10 @@ - + - + @@ -6819,10 +6819,10 @@ - + - + @@ -6845,7 +6845,7 @@ - + @@ -6859,7 +6859,7 @@ - + @@ -6988,7 +6988,7 @@ - + @@ -7002,7 +7002,7 @@ - + @@ -7145,7 +7145,7 @@ - + @@ -7231,7 +7231,7 @@ - + @@ -7244,26 +7244,26 @@ - + - - + + - + - - + + @@ -7345,7 +7345,7 @@ - + @@ -7357,7 +7357,7 @@ - + @@ -7381,7 +7381,7 @@ - + @@ -7429,10 +7429,10 @@ - + - + @@ -7516,10 +7516,10 @@ - + - + @@ -7896,13 +7896,13 @@ - + - + @@ -7914,13 +7914,13 @@ - + - + @@ -8453,7 +8453,7 @@ - + @@ -8468,7 +8468,7 @@ - + @@ -8484,7 +8484,7 @@ - + @@ -8492,7 +8492,7 @@ - + @@ -8514,7 +8514,7 @@ - + @@ -8529,7 +8529,7 @@ - + @@ -8545,7 +8545,7 @@ - + @@ -8553,7 +8553,7 @@ - + @@ -8647,7 +8647,7 @@ - + @@ -8655,14 +8655,14 @@ - + - + @@ -8670,8 +8670,8 @@ - - + + @@ -8685,7 +8685,7 @@ - + @@ -8693,8 +8693,8 @@ - - + + @@ -8811,8 +8811,8 @@ - - + + @@ -8822,8 +8822,8 @@ - - + + @@ -8858,7 +8858,7 @@ - + @@ -8910,7 +8910,7 @@ - + @@ -8918,7 +8918,7 @@ - + @@ -8934,7 +8934,7 @@ - + @@ -8942,7 +8942,7 @@ - + @@ -8950,7 +8950,7 @@ - + @@ -9150,7 +9150,7 @@ - + @@ -9628,8 +9628,8 @@ - - + + @@ -9667,14 +9667,14 @@ - + - + @@ -9694,7 +9694,7 @@ - + @@ -9773,21 +9773,21 @@ - + - + - + @@ -9873,7 +9873,7 @@ - + @@ -9937,14 +9937,14 @@ - + - + @@ -9985,7 +9985,7 @@ - + @@ -10023,7 +10023,7 @@ - + @@ -10084,7 +10084,7 @@ - + @@ -10142,7 +10142,7 @@ - + @@ -10154,7 +10154,7 @@ - + @@ -10535,7 +10535,7 @@ - + @@ -10551,7 +10551,7 @@ - + @@ -10582,7 +10582,7 @@ - + @@ -10715,15 +10715,15 @@ - - - - + + + + - + @@ -10738,7 +10738,7 @@ - + @@ -10749,8 +10749,8 @@ - - + + @@ -10761,8 +10761,8 @@ - - + + @@ -10775,14 +10775,14 @@ - - + + - + @@ -10791,8 +10791,8 @@ - - + + @@ -10804,19 +10804,19 @@ - - + + - + - + - + @@ -10830,7 +10830,7 @@ - + @@ -10844,7 +10844,7 @@ - + @@ -10861,18 +10861,18 @@ - + - + - + - + @@ -10883,10 +10883,10 @@ - + - + @@ -10928,7 +10928,7 @@ - + @@ -10952,7 +10952,7 @@ - + @@ -11005,7 +11005,7 @@ - + @@ -11029,7 +11029,7 @@ - + @@ -11168,7 +11168,7 @@ - + @@ -11181,7 +11181,7 @@ - + @@ -11189,8 +11189,8 @@ - - + + @@ -11329,7 +11329,7 @@ - + @@ -11370,7 +11370,7 @@ - + @@ -11406,7 +11406,7 @@ - + @@ -11436,7 +11436,7 @@ - + @@ -11701,7 +11701,7 @@ - + @@ -11725,7 +11725,7 @@ - + @@ -11742,7 +11742,7 @@ - + @@ -11759,7 +11759,7 @@ - + @@ -11793,7 +11793,7 @@ - + @@ -11817,7 +11817,7 @@ - + @@ -11840,7 +11840,7 @@ - + @@ -12171,7 +12171,7 @@ - + @@ -12189,8 +12189,8 @@ - - + + @@ -12213,14 +12213,14 @@ - + - + - + @@ -12235,10 +12235,10 @@ - + - + @@ -12247,20 +12247,20 @@ - - + + - + - + - + @@ -12302,11 +12302,11 @@ - + - + @@ -12327,7 +12327,7 @@ - + @@ -12339,7 +12339,7 @@ - + @@ -12364,7 +12364,7 @@ - + @@ -12373,15 +12373,15 @@ - - + + - - + + @@ -12391,42 +12391,42 @@ - - + + - - + + - + - + - + - - + + - + - - + + @@ -12436,7 +12436,7 @@ - + @@ -12447,7 +12447,7 @@ - + @@ -12458,7 +12458,7 @@ - + @@ -12469,18 +12469,18 @@ - - + + - + - + @@ -12490,30 +12490,30 @@ - + - + - + - + - + @@ -12531,35 +12531,35 @@ - + - + - + - + - + - + - + - + - + - + - + @@ -12580,7 +12580,7 @@ - + @@ -12592,7 +12592,7 @@ - + @@ -12618,7 +12618,7 @@ - + @@ -12636,7 +12636,7 @@ - + @@ -12718,28 +12718,28 @@ - + - + - - + + - + - + - + - + @@ -12759,7 +12759,7 @@ - + @@ -12767,7 +12767,7 @@ - + @@ -12786,31 +12786,31 @@ - - + + - + - - - - + + + + - - - + + + - - + + @@ -12891,14 +12891,14 @@ - + - + @@ -12906,7 +12906,7 @@ - + @@ -13080,16 +13080,16 @@ - + - + - + @@ -13268,7 +13268,7 @@ - + @@ -13448,8 +13448,8 @@ - - + + @@ -13698,7 +13698,7 @@ - + @@ -13706,7 +13706,7 @@ - + @@ -13978,7 +13978,7 @@ - + @@ -13994,14 +13994,14 @@ - + - + @@ -14009,7 +14009,7 @@ - + @@ -14036,7 +14036,7 @@ - + @@ -14044,7 +14044,7 @@ - + @@ -14067,7 +14067,7 @@ - + @@ -14075,7 +14075,7 @@ - + @@ -14208,7 +14208,7 @@ - + @@ -14216,7 +14216,7 @@ - + @@ -14224,7 +14224,7 @@ - + @@ -14233,14 +14233,14 @@ - + - + @@ -14327,11 +14327,11 @@ - + - + @@ -14435,7 +14435,7 @@ - + @@ -14514,8 +14514,8 @@ - - + + @@ -14530,16 +14530,16 @@ - - + + - - + + @@ -14570,7 +14570,7 @@ - + @@ -14617,11 +14617,11 @@ - + - + - + @@ -15108,7 +15108,7 @@ - + @@ -15158,13 +15158,13 @@ - + - + @@ -15183,25 +15183,25 @@ - + - + - + - + - + @@ -15223,14 +15223,14 @@ - + - + @@ -15242,16 +15242,16 @@ - + - + - + @@ -15322,7 +15322,7 @@ - + @@ -15533,7 +15533,7 @@ - + @@ -16478,7 +16478,7 @@ - + @@ -16518,7 +16518,7 @@ - + @@ -16583,14 +16583,14 @@ - + - + @@ -16688,14 +16688,14 @@ - + - + @@ -16721,7 +16721,7 @@ - + @@ -17420,7 +17420,7 @@ - + @@ -17455,7 +17455,7 @@ - + @@ -17477,7 +17477,7 @@ - + @@ -17485,15 +17485,15 @@ - + - - + + @@ -17634,13 +17634,13 @@ - + - + @@ -17648,7 +17648,7 @@ - + @@ -17677,7 +17677,7 @@ - + @@ -17685,7 +17685,7 @@ - + @@ -17693,20 +17693,20 @@ - + - + - + @@ -17788,20 +17788,20 @@ - + - - + + - + @@ -17812,14 +17812,14 @@ - + - - + + @@ -17878,8 +17878,8 @@ - - + + @@ -18406,7 +18406,7 @@ - + @@ -18414,7 +18414,7 @@ - + @@ -18424,7 +18424,7 @@ - + @@ -18436,7 +18436,7 @@ - + @@ -18599,7 +18599,7 @@ - + @@ -18706,13 +18706,13 @@ - + - + @@ -18768,16 +18768,16 @@ - + - + - + @@ -18786,30 +18786,30 @@ - + - + - + - + - + - + - - + + @@ -18944,8 +18944,8 @@ - - + + @@ -19034,8 +19034,8 @@ - - + + @@ -19049,7 +19049,7 @@ - + @@ -19587,14 +19587,14 @@ - + - + @@ -19757,7 +19757,7 @@ - + @@ -19765,8 +19765,8 @@ - - + + @@ -19781,8 +19781,8 @@ - - + + @@ -20015,13 +20015,13 @@ - + - - + + @@ -20043,8 +20043,8 @@ - - + + @@ -20222,13 +20222,13 @@ - + - + @@ -20239,7 +20239,7 @@ - + @@ -20425,7 +20425,7 @@ - + @@ -20477,7 +20477,7 @@ - + @@ -20529,7 +20529,7 @@ - + @@ -20581,7 +20581,7 @@ - + @@ -20633,7 +20633,7 @@ - + @@ -20679,7 +20679,7 @@ - + @@ -20687,7 +20687,7 @@ - + @@ -20695,7 +20695,7 @@ - + @@ -20752,7 +20752,7 @@ - + @@ -20818,7 +20818,7 @@ - + @@ -20880,7 +20880,7 @@ - + @@ -20919,7 +20919,7 @@ - + @@ -20949,10 +20949,10 @@ - + - - + + @@ -20989,8 +20989,8 @@ - - + + @@ -21078,11 +21078,11 @@ - - + + - + @@ -21094,24 +21094,24 @@ - + - + - + - + - - + + - + @@ -21131,37 +21131,37 @@ - + - + - + - + - + - + - + - + - + - + - + @@ -21206,7 +21206,7 @@ - + @@ -21254,92 +21254,92 @@ - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + - + @@ -21371,7 +21371,7 @@ - + @@ -21398,8 +21398,8 @@ - - + + @@ -21422,10 +21422,10 @@ - - + + - + @@ -21437,15 +21437,15 @@ - + - - + + @@ -21500,7 +21500,7 @@ - + @@ -21803,7 +21803,7 @@ - + @@ -21811,7 +21811,7 @@ - + @@ -21833,7 +21833,7 @@ - + @@ -22070,7 +22070,7 @@ - + @@ -22081,13 +22081,13 @@ - + - + @@ -22159,44 +22159,44 @@ - + - + - + - + - + - + - + - + - + - + - + - + @@ -22207,24 +22207,24 @@ - + - + - + - + - + - + @@ -22235,10 +22235,10 @@ - + - + @@ -22249,10 +22249,10 @@ - + - + @@ -22263,10 +22263,10 @@ - + - + @@ -22311,7 +22311,7 @@ - + @@ -22320,14 +22320,14 @@ - + - + @@ -22335,14 +22335,14 @@ - + - + diff --git a/tests/data/test-read-dwarf/test18-pr19037-libvtkRenderingLIC-6.1.so.abi b/tests/data/test-read-dwarf/test18-pr19037-libvtkRenderingLIC-6.1.so.abi index 4b385657..698dcce2 100644 --- a/tests/data/test-read-dwarf/test18-pr19037-libvtkRenderingLIC-6.1.so.abi +++ b/tests/data/test-read-dwarf/test18-pr19037-libvtkRenderingLIC-6.1.so.abi @@ -12933,16 +12933,7 @@ - - - - - - - - - - + @@ -12950,6 +12941,15 @@ + + + + + + + + + @@ -14796,16 +14796,7 @@ - - - - - - - - - - + @@ -14813,6 +14804,15 @@ + + + + + + + + + diff --git a/tests/data/test-read-dwarf/test22-pr19097-libstdc++.so.6.0.17.so.abi b/tests/data/test-read-dwarf/test22-pr19097-libstdc++.so.6.0.17.so.abi index e46bbec1..2ade2cff 100644 --- a/tests/data/test-read-dwarf/test22-pr19097-libstdc++.so.6.0.17.so.abi +++ b/tests/data/test-read-dwarf/test22-pr19097-libstdc++.so.6.0.17.so.abi @@ -22274,20 +22274,20 @@ - - - - - - - + + + + + + + @@ -22540,7 +22540,7 @@ - + @@ -41532,7 +41532,7 @@ - + @@ -41641,20 +41641,20 @@ - - - - - - - + + + + + + + @@ -41907,7 +41907,7 @@ - + @@ -48396,15 +48396,7 @@ - - - - - - - - - + @@ -48413,6 +48405,14 @@ + + + + + + + + @@ -52359,15 +52359,15 @@ - + - + - + - + From patchwork Wed Oct 4 10:54:55 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dodji Seketeli X-Patchwork-Id: 77091 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 71EB5385841D for ; Wed, 4 Oct 2023 10:55:12 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 71EB5385841D DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1696416912; bh=e5Ok8bsJB6W7zCw6LSEIfFxqdegYrWM1Z8Y0fbVkwIY=; h=To:Subject:References:Date:In-Reply-To:List-Id:List-Unsubscribe: List-Archive:List-Help:List-Subscribe:From:Reply-To:Cc:From; b=ZD37LKwAM2xVDBv1GGNFu0Ua/G6SKQH9X1eLBaFKTbjAxH27Za6OTGPkCtpmRW+pC setAkY22M8SbPQciotfK50sOU+aLHjOkXHta7+PWzjgb39G1YQG2AGUVpKEePrFVa5 zJ/5is9DFFxrZmxZhUaPZawJ2aB6MWCGCZTNQejc= X-Original-To: libabigail@sourceware.org Delivered-To: libabigail@sourceware.org Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTPS id 2103F385841B for ; Wed, 4 Oct 2023 10:55:05 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 2103F385841B Received: from mail-qt1-f198.google.com (mail-qt1-f198.google.com [209.85.160.198]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-163-U1kW0nO3NZuoKYaIi-IUjQ-1; Wed, 04 Oct 2023 06:54:58 -0400 X-MC-Unique: U1kW0nO3NZuoKYaIi-IUjQ-1 Received: by mail-qt1-f198.google.com with SMTP id d75a77b69052e-418198baf49so21730421cf.1 for ; Wed, 04 Oct 2023 03:54:58 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1696416897; x=1697021697; h=mime-version:user-agent:message-id:in-reply-to:date:references :organization:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=e5Ok8bsJB6W7zCw6LSEIfFxqdegYrWM1Z8Y0fbVkwIY=; b=FU0X+51GKPyQYLfskgsOqqqy2hEi0IYw9BriZYkclLJDBChMJBV8q7JVs0NrKq78GD vHkpSyKy/ivaIgQ7ojn/2Xsa8EX5nKKJ0aX8z5XoclD/dXPzik/jUItOVt/GR9hzoTBW wfVXmMY5eg4WSo+2LVK1WXAw8m/pBN0HypqLJvD5MzKPCMEAc+SXR2CdI6MADvjGobPt pG0Dm/yz+1CzB5JINGcfPgt0nHd0WAi2AQV4XfIYUEFGi4hKVthwb10POPmxGxbExhP8 P8HiyF7RlRrISNJk2V8BZjUhwU7tDOtecWVcdZOhVEuzOzM7vBcfj2cMLAJhqBgD6vMb +tag== X-Gm-Message-State: AOJu0Yz8ByREHiSJhdTC4sOMscbQ12p82ey1WCEhXwZVXZkrb3HQK0nA 6+Jk6vljcKUxAhNjyxKGGppZOol5+f0DDX/CqERtgm0Gwdq0NKMHPKQ4kcbaDzfJCZlFOS3glWt 6dMhywWHKOqntq5OJJpB1aRuXQXJs X-Received: by 2002:a05:622a:1913:b0:413:5f30:9995 with SMTP id w19-20020a05622a191300b004135f309995mr2367398qtc.36.1696416897499; Wed, 04 Oct 2023 03:54:57 -0700 (PDT) X-Google-Smtp-Source: AGHT+IERp1WR0GllSdFepr2QgWhlFlagIF52a8mIyh0iRSZD06ep5Hpgb1ANTOI4eQ9Ol/h4yvel5w== X-Received: by 2002:a05:622a:1913:b0:413:5f30:9995 with SMTP id w19-20020a05622a191300b004135f309995mr2367383qtc.36.1696416897154; Wed, 04 Oct 2023 03:54:57 -0700 (PDT) Received: from localhost ([88.120.130.27]) by smtp.gmail.com with ESMTPSA id h13-20020ac8514d000000b004196d75d79csm1125246qtn.46.2023.10.04.03.54.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 04 Oct 2023 03:54:56 -0700 (PDT) Received: by localhost (Postfix, from userid 1000) id 313A3B6EB1; Wed, 4 Oct 2023 12:54:55 +0200 (CEST) To: Dodji Seketeli Subject: [PATCH 3/3, V2] For WIP branch check-uapi-support: ir,comparison: Represent changed anonymous enums Organization: Red Hat / France References: <87r0mapspt.fsf@redhat.com> X-Operating-System: CentOS Stream release 9 X-URL: http://www.redhat.com Date: Wed, 04 Oct 2023 12:54:55 +0200 In-Reply-To: <87r0mapspt.fsf@redhat.com> (Dodji Seketeli's message of "Wed, 04 Oct 2023 12:50:06 +0200") Message-ID: <87cyxupshs.fsf@redhat.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Spam-Status: No, score=-11.9 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H4, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: libabigail@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Mailing list of the Libabigail project List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-Patchwork-Original-From: Dodji Seketeli via Libabigail From: Dodji Seketeli Reply-To: Dodji Seketeli Cc: Trilok Soni , libabigail@sourceware.org, Satya Durga Srinivasu Prabhala Errors-To: libabigail-bounces+patchwork=sourceware.org@sourceware.org Sender: "Libabigail" Hello, Now that added/removed non-reachable anonymous enums is supported, we want to represent changing an anonymous enum. Strictly speaking, adding or removing an enumerator from an anonymous enum is represented as deleting of the anonymous enum in the old state and the addition of an anonymous enum in the new state. This patch analyses the added/removed anonymous enums and if the old enum has enumerators contained in the new one, then it assumes we are looking at anonymous enum change. * include/abg-ir.h (is_enumerator_present_in_enum): Declare new public function. * src/abg-ir.cc (is_enumerator_present_in_enum): Turn this static function into a public one. * src/abg-comparison.cc (corpus_diff::priv::ensure_lookup_tables_populated): Detect that an removed/added anonymous enum is actually a changed anonymous enum and represent it as such. * tests/data/test-abidiff-exit/test-anonymous-enums-change-report-v{0,1}.txt: New reference test output files. * tests/data/test-abidiff-exit/test-anonymous-enums-change-v{0,1}.c: Source code for the some input test binary. * tests/data/test-abidiff-exit/test-anonymous-enums-change-v{0,1}.o: New test input binaries. * tests/data/Makefile.am: Add the new test materials above to source distribution. * tests/test-abidiff-exit.cc (in_out_specs): Add the new test input to the harness. Signed-off-by: Dodji Seketeli --- include/abg-ir.h | 4 + src/abg-comparison.cc | 76 ++++++++++++++++++ src/abg-ir.cc | 2 +- tests/data/Makefile.am | 6 ++ .../test-anonymous-enums-change-report-v0.txt | 16 ++++ .../test-anonymous-enums-change-report-v1.txt | 21 +++++ .../test-anonymous-enums-change-v0.c | 36 +++++++++ .../test-anonymous-enums-change-v0.o | Bin 0 -> 3296 bytes .../test-anonymous-enums-change-v1.c | 41 ++++++++++ .../test-anonymous-enums-change-v1.o | Bin 0 -> 3336 bytes tests/test-abidiff-exit.cc | 32 ++++++++ 11 files changed, 233 insertions(+), 1 deletion(-) create mode 100644 tests/data/test-abidiff-exit/test-anonymous-enums-change-report-v0.txt create mode 100644 tests/data/test-abidiff-exit/test-anonymous-enums-change-report-v1.txt create mode 100644 tests/data/test-abidiff-exit/test-anonymous-enums-change-v0.c create mode 100644 tests/data/test-abidiff-exit/test-anonymous-enums-change-v0.o create mode 100644 tests/data/test-abidiff-exit/test-anonymous-enums-change-v1.c create mode 100644 tests/data/test-abidiff-exit/test-anonymous-enums-change-v1.o new file mode 100644 index 00000000..886dbd2b index fb3d6fff..4c1eaea5 100644 diff --git a/include/abg-ir.h b/include/abg-ir.h index 3e2cbda9..d21cd2fa 100644 --- a/include/abg-ir.h +++ b/include/abg-ir.h @@ -2828,6 +2828,10 @@ public: set_enum_type(enum_type_decl*); }; // end class enum_type_def::enumerator +bool +is_enumerator_present_in_enum(const enum_type_decl::enumerator &enr, + const enum_type_decl &enom); + bool equals(const typedef_decl&, const typedef_decl&, change_kind*); diff --git a/src/abg-comparison.cc b/src/abg-comparison.cc index 228909dc..eb8e30ac 100644 --- a/src/abg-comparison.cc +++ b/src/abg-comparison.cc @@ -14,6 +14,7 @@ #include #include #include +#include #include "abg-comparison-priv.h" #include "abg-reporter-priv.h" @@ -9609,6 +9610,81 @@ corpus_diff::priv::ensure_lookup_tables_populated() added_unreachable_types_[repr] = t; } } + + // Handle anonymous enums that got changed. An anonymous enum is + // designated by its flat textual representation. So a change to + // any of its enumerators results in a different enum. That is + // represented by a deletion of the previous anonymous enum, and + // the addition of a new one. For the user however, it's the same + // enum that changed. Let's massage this "added/removed" pattern + // to show what the user expects, namely, a changed anonymous + // enum. + { + std::set deleted_anon_enums; + std::set added_anon_enums; + + for (auto entry : deleted_unreachable_types_) + if (is_enum_type(entry.second) + && is_enum_type(entry.second)->get_is_anonymous()) + deleted_anon_enums.insert(is_enum_type(entry.second)); + + for (auto entry : added_unreachable_types_) + if (is_enum_type(entry.second) + && is_enum_type(entry.second)->get_is_anonymous()) + added_anon_enums.insert(is_enum_type(entry.second)); + + string_type_base_sptr_map added_anon_enum_to_erase; + string_type_base_sptr_map removed_anon_enum_to_erase; + + // Look for deleted anonymous enums which have enumerators + // present in an added anonymous enums ... + for (auto deleted_enum : deleted_anon_enums) + { + // Look for any enumerator of 'deleted_enum' that is also + // present in an added anonymous enum. + for (auto enr : deleted_enum->get_enumerators()) + { + bool this_enum_got_changed = false; + for (auto added_enum : added_anon_enums) + { + if (is_enumerator_present_in_enum(enr, *added_enum)) + { + // So the enumerator 'enr' from the + // 'deleted_enum' enum is also present in the + // 'added_enum' enum so we assume that + // 'deleted_enum' and 'added_enum' are the same + // enum that got changed. Let's represent it + // using a diff node. + diff_sptr d = compute_diff(deleted_enum, + added_enum, ctxt); + ABG_ASSERT(d->has_changes()); + string repr = + abigail::ir::get_pretty_representation(is_type(deleted_enum), + /*internal=*/false); + changed_unreachable_types_[repr]= d; + this_enum_got_changed = true; + string r1 = abigail::ir::get_pretty_representation(is_type(deleted_enum)); + string r2 = abigail::ir::get_pretty_representation(is_type(added_enum)); + removed_anon_enum_to_erase[r1] = deleted_enum; + added_anon_enum_to_erase[r2] = added_enum; + break; + } + } + if (this_enum_got_changed) + break; + } + } + + // Now remove the added/removed anonymous enums from their maps, + // as they are now represented as a changed enum, not an added + // and removed enum. + + for (auto entry : added_anon_enum_to_erase) + added_unreachable_types_.erase(entry.first); + + for (auto entry : removed_anon_enum_to_erase) + deleted_unreachable_types_.erase(entry.first); + } } } diff --git a/src/abg-ir.cc b/src/abg-ir.cc index 06dfbe20..171267f8 100644 --- a/src/abg-ir.cc +++ b/src/abg-ir.cc @@ -18997,7 +18997,7 @@ enum_has_non_name_change(const enum_type_decl& l, /// /// @return true iff the enumerator @p enr is present in the enum @p /// enom. -static bool +bool is_enumerator_present_in_enum(const enum_type_decl::enumerator &enr, const enum_type_decl &enom) { diff --git a/tests/data/Makefile.am b/tests/data/Makefile.am index 0569af34..d1c05ff7 100644 --- a/tests/data/Makefile.am +++ b/tests/data/Makefile.am @@ -356,6 +356,12 @@ test-abidiff-exit/test-enumerator-changes1-v0.c \ test-abidiff-exit/test-enumerator-changes1-v0.o \ test-abidiff-exit/test-enumerator-changes1-v1.c \ test-abidiff-exit/test-enumerator-changes1-v1.o \ +test-abidiff-exit/test-anonymous-enums-change-report-v0.txt \ +test-abidiff-exit/test-anonymous-enums-change-report-v1.txt \ +test-abidiff-exit/test-anonymous-enums-change-v0.c \ +test-abidiff-exit/test-anonymous-enums-change-v0.o \ +test-abidiff-exit/test-anonymous-enums-change-v1.c \ +test-abidiff-exit/test-anonymous-enums-change-v1.o \ \ test-diff-dwarf/test0-v0.cc \ test-diff-dwarf/test0-v0.o \ diff --git a/tests/data/test-abidiff-exit/test-anonymous-enums-change-report-v0.txt b/tests/data/test-abidiff-exit/test-anonymous-enums-change-report-v0.txt new file mode 100644 index 00000000..2a408ea0 --- /dev/null +++ b/tests/data/test-abidiff-exit/test-anonymous-enums-change-report-v0.txt @@ -0,0 +1,16 @@ +Functions changes summary: 0 Removed, 0 Changed, 0 Added function +Variables changes summary: 0 Removed, 0 Changed, 0 Added variable +Unreachable types summary: 0 removed, 1 changed (1 filtered out), 1 added types + +1 changed type unreachable from any public interface: + + [C] 'enum {E4_0=0, E4_1=1, E4_2=2, E4_LAST_ELEMENT=3, }' changed: + type size hasn't changed + 2 enumerator deletions: + 'E4_2' value '2' + 'E4_LAST_ELEMENT' value '3' + +1 added type unreachable from any public interface: + + [A] 'enum {E5_0=0, E5_1=1, }' at test-anonymous-enums-change-v1.c:26:1 + diff --git a/tests/data/test-abidiff-exit/test-anonymous-enums-change-report-v1.txt b/tests/data/test-abidiff-exit/test-anonymous-enums-change-report-v1.txt new file mode 100644 index 00000000..8639fa44 --- /dev/null +++ b/tests/data/test-abidiff-exit/test-anonymous-enums-change-report-v1.txt @@ -0,0 +1,21 @@ +Functions changes summary: 0 Removed, 0 Changed, 0 Added function +Variables changes summary: 0 Removed, 0 Changed, 0 Added variable +Unreachable types summary: 0 removed, 2 changed, 1 added types + +2 changed types unreachable from any public interface: + + [C] 'enum {E4_0=0, E4_1=1, E4_2=2, E4_LAST_ELEMENT=3, }' changed: + type size hasn't changed + 2 enumerator deletions: + 'E4_2' value '2' + 'E4_LAST_ELEMENT' value '3' + + [C] 'enum {E3_0=0, E3_1=1, }' changed: + type size hasn't changed + 1 enumerator insertion: + 'E3_2' value '2' + +1 added type unreachable from any public interface: + + [A] 'enum {E5_0=0, E5_1=1, }' at test-anonymous-enums-change-v1.c:26:1 + diff --git a/tests/data/test-abidiff-exit/test-anonymous-enums-change-v0.c b/tests/data/test-abidiff-exit/test-anonymous-enums-change-v0.c new file mode 100644 index 00000000..443eb971 --- /dev/null +++ b/tests/data/test-abidiff-exit/test-anonymous-enums-change-v0.c @@ -0,0 +1,36 @@ +/* + * Compile this with: + * gcc -c -g -fno-eliminate-unused-debug-types test-anonymous-enums-change-v0.c + */ + +enum + { + E1_0, + E1_1, + } v1; + +enum + { + E3_0, + E3_1, + }; + +enum + { + E4_0, + E4_1, + E4_2, + E4_LAST_ELEMENT + }; + +enum + { + E0_0, + E0_1, + } v0; + +enum + { + E2_0, + E2_1, + } v2; diff --git a/tests/data/test-abidiff-exit/test-anonymous-enums-change-v0.o b/tests/data/test-abidiff-exit/test-anonymous-enums-change-v0.o new file mode 100644 index 0000000000000000000000000000000000000000..26c6df80f0d5203c520890f433cd5bba20b0bc4f GIT binary patch literal 3296 zcmb_e&2Jl35P$2nNn4Y&sU^Nrl6BRJ1laZ3aYJd9U=*3O5Gn*kLR_4+{p>8*>)>6R zL})`WoOXA z9GAtpMG(%Ks3NI=*$zm@0u)^MSc8>IMy+NXx$uSN0ySK*h=E#-;?L8b6>;U;W1hlT z7UtZgJnSoLY!qkm#q90EjiPZg|7_7T3h)M1VV)r%cf0W1qnSs9$j%#A=L&gqX<_NY z!*DuRL_WWfH|HSrC3rIM2$48WrN>JJSPpDY0yhlfuen)d_=fO+utcxClsO-#QDLJX z^t*v9`YNicpl;oH-P$Za zW0lKJ)hSyQx3cQ4x!0`aS0Vc?FCv3oS+VSXGz#Q;R|azEw?OrVt=;wg>udH})gD(Y zyKC8m!VawIQNx*ap>x6r!UwTS#kgGG^?3CtiTC0+xk@CfjS52zX3iL|G7!g~137t0yhFtCG+u-U>0-0~3((?n!Ay9%vgd0k44f=fvKRr$`h@=Cv%nl>3#c#$N{%?E%!tiYQ zJUL5^1&yO`lCVDz(6DQ5_$R=I_v)m0eKJkv1m(oVfJU+fFa zS(|vo!VWdnpA>+-hkiT*;D3msby)SBXjmlh73Fob=)P#zG4u`yv`|Tq?)wth()Rxb z{WS1CasLSE`)>oL=LAT6N+8b*#E^8~24HFR_o)6Y6&2rIoMU?ZJ<92lAG1R$bjSd4 zuFw2Ez){xYKgz@be^4AVj${4_7}DbJQA6@PKPevPBIjZsH3f1ireO;)}B{0Z^j)A##w5*Z76{6Aj89Torp literal 0 HcmV?d00001 diff --git a/tests/data/test-abidiff-exit/test-anonymous-enums-change-v1.c b/tests/data/test-abidiff-exit/test-anonymous-enums-change-v1.c --- /dev/null +++ b/tests/data/test-abidiff-exit/test-anonymous-enums-change-v1.c @@ -0,0 +1,41 @@ +/* + * Compile this with: + * gcc -c -g -fno-eliminate-unused-debug-types test-anonymous-enums-change-v1.c + */ + +enum + { + E1_0, + E1_1, + } v1; + +enum + { + E3_0, + E3_1, + E3_2 + }; + +enum + { + E4_0, + E4_1, + }; + +enum + { + E5_0, + E5_1, + }; + +enum + { + E0_0, + E0_1, + } v0; + +enum + { + E2_0, + E2_1, + } v2; diff --git a/tests/data/test-abidiff-exit/test-anonymous-enums-change-v1.o b/tests/data/test-abidiff-exit/test-anonymous-enums-change-v1.o new file mode 100644 index 0000000000000000000000000000000000000000..7d0f71b47a47d0a7eb893ddd41c4515cfb4b28fa GIT binary patch literal 3336 zcmb_eO>7%g5T5ngq^(JsG>D(JkTtX=q3o``aY6}7YAFqA1*#Ad7cORPKRXNdI`Xbf zB7|Cr3j&D?91s^SaN>jnhe`->=$%u!aOsH)2gHd32bg&~FWEeAFhY#9Z)U#vdprBy z*tfP`elcelpvZs?IMfsh@cxNIxhBOLoPjy`73BgL(wZmyacSfww}QeHElEzf8c@hj zk%Kv^z^NxtIBAkfBm<_qD5V3WoJ))ky{8!y`0Qtyu=TYLo1-#3Mb28$OFxf$da!i& zh$k3}V9uN`;zskvMro>8%HNn@D;X~opD&rlJibyD%o7Y0Zp>eOEcYlte$Kc!GhZ|p z&MnM7g6+Bh#o|WMoWa!R@yX!>km|40lf^?;K-)9O4a4|zW||oF10NI?lGYdtxzkA+ z88!@~pc9Ig6@)QtRqAD0DzH^1X(d^tic6K$Dp{9FT23Snwr{>>ZC0MMDix>dR4lLT zEtglzE7szxnD@FLv%&V3EV~yEL$Tfwp%?^BRQ*A7XMOL=s=ZpZN1kPOEV~`{ZP5*S zLFmWA9)`n6wCt8>3_Etbzbhix@~FIrF&Rjgtw}W8jmB%%$E-y~#z!m9F?ABMGv!}@Dl51Q@E-{&{7Jg{HmU-;i}#h4QCa$ zS)lW!auD=7<7)pj^NHS&YqEE$yT*E3!>_XbwuWD3{ap<&v;L8W(?BKqBw?~=R9E%q zOAW_)Ea@u^zr^}?%;|m8r|U<}{~OltY4{!1e_>8}PP0r*odN1wA%)ZVBvRk^twe=q znJfHT)+-wR6YJna`@Pt2Add$L-%(O5_F`~oG=kHJB5($x>pOBN*MqR#M;i_3y5DFF z#OR3GAJ8~H8qy6yp+c~P5(%zLz1i>e1pXp9VLuiQ4mvxE{ib>|wc-E9&tVw#R`pY5 zsVPC@1eaWt4