From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id kZ+cFVTwVGohPQYAWB0awg (envelope-from ) for ; Mon, 13 Jul 2026 10:04:04 -0400 Authentication-Results: simark.ca; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=amd.com header.i=@amd.com header.a=rsa-sha256 header.s=selector1 header.b=SiMOjqqs; dkim-atps=neutral Received: by simark.ca (Postfix, from userid 112) id 531321E09B; Mon, 13 Jul 2026 10:04:04 -0400 (EDT) X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-25) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-5.1 required=5.0 tests=ARC_SIGNED,ARC_VALID,BAYES_00, DKIM_INVALID,DKIM_SIGNED,MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED autolearn=ham autolearn_force=no version=4.0.1 Received: from vm01.sourceware.org (vm01.sourceware.org [38.145.34.32]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id 614201E033 for ; Mon, 13 Jul 2026 10:04:03 -0400 (EDT) Received: from vm01.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id EE6E34BA23CB for ; Mon, 13 Jul 2026 14:04:02 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org EE6E34BA23CB Authentication-Results: sourceware.org; dkim=fail reason="signature verification failed" (1024-bit key, unprotected) header.d=amd.com header.i=@amd.com header.a=rsa-sha256 header.s=selector1 header.b=SiMOjqqs Received: from PH8PR06CU001.outbound.protection.outlook.com (mail-westus3azon11012016.outbound.protection.outlook.com [40.107.209.16]) by sourceware.org (Postfix) with ESMTPS id 4956D4BA2E1B for ; Mon, 13 Jul 2026 14:00:55 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 4956D4BA2E1B Authentication-Results: sourceware.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: sourceware.org; spf=fail smtp.mailfrom=amd.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 4956D4BA2E1B Authentication-Results: sourceware.org; arc=pass smtp.remote-ip=40.107.209.16 ARC-Seal: i=2; a=rsa-sha256; d=sourceware.org; s=key; t=1783951255; cv=pass; b=rvRSY0GVwFr5ggHeM7bJNz5nzUII2KDxrOdm7Q1d8ax0UxQvGQiH6Vw3f6TZu/FYq3Zthwjn34voOY2syXA3wXXblxl9irpLjirtJ6PnWVTQgvwmnLqDZQyJe/gPLk5A0Exa1U/MLlcbvfT3TjDoVdzht4spk5fKi8ujDIdMbnc= ARC-Message-Signature: i=2; a=rsa-sha256; d=sourceware.org; s=key; t=1783951255; c=relaxed/simple; bh=/plb8tl8x8uymPv1yf2OsicWDK1xUdgagNaN3+AnN5o=; h=DKIM-Signature:From:To:Subject:Date:Message-ID:MIME-Version; b=OFKVm5IK9CeJ8CjmdBp3OCkkfkbOHrQTH3tKx/dnSdfj7krOhsUTgu3PgpoH29T3+JeqjAFjdbY6yF6Ml/LadSQjBBJExjqSL7fMSZY1bjV00CCY0lboqMMRCMZuXg5mSth4zbNtNCuymS+DM054Ypxgx+rH6BrRs/OmKvypyz0= ARC-Authentication-Results: i=2; sourceware.org; dkim=pass (1024-bit key, unprotected) header.d=amd.com header.i=@amd.com header.a=rsa-sha256 header.s=selector1 header.b=SiMOjqqs DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 4956D4BA2E1B ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=mt/8EkLhMBm2E0xIX6ifl7IgW44gkk0wx1R5zOc/k9YFIxg/UXBMsrZXOJ21U4PREEiZ0HwWg+pLXTfgOoNhwTF/f3Gb/Lb+NuPx0O/cCtgBwdGWxAr5lMlSgnfnuFp+lIAWuTNLdxzY2/c5IHTQoc2fc8i99tOEQSEasZCZifyQAHsutxhBHmzVsPl29/iTH2LnE2JODDUGcyELWjw6MzDDFdZPsWKpruTK70VSa3WZUO47j0uxGzaeuU5glvoozahjS+vDXASd8AAtNcOrObLtOJV1zdiKicJFXXxcstRGxNnua0mqPDTWWsPtywrYmpQLystKMjUoenV2tI2K2Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=cKx0+bJlvq9zu/zdZCR4sDzHPe2WSHD2Ko2eIoQ/8Sc=; b=bdPVxdGGgrMPzGyNrJ1LBWd4t1faeMtB9aDa/0k4SW62bsRJX8g1bloJ8USsfNWrkcKkFN0Kmqtk/5p/6GEpQMKbP+pA/LP9OihDAf6nQx7jvIEY2/jd13TzibfmflAJthWajP4Mux4XXeHY8WeHeh6DRW+f+2elVTT1UEIxsfuzG0oPLpNKSxntGZJbdXnlmm+xJRRxVLu4zoHaxDYFzSRCD04qrbGaef11oyYq6vDjcZ8UsXQ/W4tqBh1hJM3yRdscC25ScWnow+dnctHzSNyFFNYM1Ks3t4kP5i9Z9DY3b+ul1uT6rGdl7wOwSAFsUrRKvAEs86JObiS+cnB8yA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=sourceware.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=cKx0+bJlvq9zu/zdZCR4sDzHPe2WSHD2Ko2eIoQ/8Sc=; b=SiMOjqqsvV2msPnacTElbf+ikNbx1eVNX/THAw69oX//qARMvXXVSz+A/CbNYwmNSVFuhJoJknEvXTPTw/cf0FHBI4TVS9UvNFQ4pbuZtZbPvzY5F2NJUKuT988XefOzk1SGe+RMYQMxe5VlXROKMJIfrjPgotTY1kP31t7b430= Received: from CH0P221CA0012.NAMP221.PROD.OUTLOOK.COM (2603:10b6:610:11c::16) by PH8PR12MB7254.namprd12.prod.outlook.com (2603:10b6:510:225::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.21.202.19; Mon, 13 Jul 2026 14:00:49 +0000 Received: from CH1PEPF0000AD7C.namprd04.prod.outlook.com (2603:10b6:610:11c:cafe::96) by CH0P221CA0012.outlook.office365.com (2603:10b6:610:11c::16) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.21.202.19 via Frontend Transport; Mon, 13 Jul 2026 14:00:48 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=satlexmb07.amd.com; pr=C Received: from satlexmb07.amd.com (165.204.84.17) by CH1PEPF0000AD7C.mail.protection.outlook.com (10.167.244.84) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.21.223.9 via Frontend Transport; Mon, 13 Jul 2026 14:00:48 +0000 Received: from ctr-rack32-mi300x-1.amd.com (10.180.168.240) by satlexmb07.amd.com (10.181.42.216) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.41; Mon, 13 Jul 2026 09:00:40 -0500 From: Tankut Baris Aktemur To: Subject: [PATCH 05/16] gdb: inline address_space_{name, type_instance_flags}_to_{type_instance_flags, name} Date: Mon, 13 Jul 2026 09:00:04 -0500 Message-ID: <20260713-users-aktemur-type-instance-flags-v1-5-779cad0c85ec@amd.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20260713-users-aktemur-type-instance-flags-v1-0-779cad0c85ec@amd.com> References: <20260713-users-aktemur-type-instance-flags-v1-0-779cad0c85ec@amd.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" X-Mailer: b4 0.15.0 Content-Transfer-Encoding: 8bit X-Originating-IP: [10.180.168.240] X-ClientProxiedBy: satlexmb08.amd.com (10.181.42.217) To satlexmb07.amd.com (10.181.42.216) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH1PEPF0000AD7C:EE_|PH8PR12MB7254:EE_ X-MS-Office365-Filtering-Correlation-Id: 6941e974-5d29-43a5-9c94-08dee0e7247d X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230040|1800799024|23010399003|82310400026|376014|36860700016|18002099003|22082099003|6133799003|56012099006|11063799006|3023799007; X-Microsoft-Antispam-Message-Info: HaABZr42lo8YjEqwsl3oVHEs0qCoBPWhM/Ua8yOnJnAdqg9z9aD3Swtctjs/9D3kmr8gEwqjqSc7/4cHDCNmgt9dP5UfKX8UWLUYTYHF0mCsyTTAQbsvvq7ixwkXrZhby83mRo34g99ASTcF3Qc2jVSyMBxQNYafR71CPUosCepfLvYau2MS7uKAGEVDK+nq5PvRkF4u8A3it/gBWXCYeG3uuXhrJOAX8eCnQJ7cf0dKopp2VAuDTSR3p6Jgz7hSd+FNK+DhM3I/YTavJ+HBJklBMUHPugmTIxoDSwfQ+9hy9XGeXdQfrADp2awduF8Mj5Y9/sjoq99exC3RS98gDkRWrpf07mkNW+dtso1Ya0rlZ+EjLOfbJzQqCjNcRjXrSTgMd6lj9Nb97sXoiBrK7LDlFge4pQ1e8c3F1sirpON3+LErqQWxXBXf0KuQ+v9dUp9feD712xwp22mzfPzmHxQ3kG6n0KdSmRp5WrN3gUbUQObSLmizFr3olo5AOn1VOQpkOYz2bnpkoy8A+hL08so3/N/9u5A/Z3hIxn8FHv5TfUG4HwjkB1d03F6EiKW+MI/9oMxf9LXLULKx6yjPJWMnFnMgO7gIGq8jn7kt5uoAm5/E3C+ufA9Yt0SVezTTeXkbdMs4eZdmxxBPQgGElbvrOfv/FQaa00mzVk192CceA/Ptw/k9PzqHcWLmR84Rwuj0hamcwASBlN4pLbu32w== X-Forefront-Antispam-Report: CIP:165.204.84.17; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:satlexmb07.amd.com; PTR:InfoDomainNonexistent; CAT:NONE; SFS:(13230040)(1800799024)(23010399003)(82310400026)(376014)(36860700016)(18002099003)(22082099003)(6133799003)(56012099006)(11063799006)(3023799007); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: 8Tmpl178ns3ZPRyIozcl7z1R8IsHN+e0fhEXo7aqSoSE1DP5ze32NMIcGok9dxT/2bSVFP3i8Xp3RMGYtOTnga/F1SWzlMzjG/hDYO/mG/+k+T1e95pzyOqGDgXaCvdEaeOZupyhE+dlyExPL889KXF0lTPHM9F3oWKncxjJZORUT9wkXJZKXBLFaNuwrzTGWJiQ4KzvXMkEGNDYPVmnHZUIVzvE9B+O2YWWcDjxx5O14j+NgbC1r9dTNqZtveAOZJNsvG+HqQuZ9LhlvVgAb6Z+qt2cv7mGIaDqFyMHLvub8w8iWk5utnajWdu5XzpoicU/W1eUkVJQMo2vuHEygaDja+jhwE6QXx+fzkzTPzOF/34wcT4wuWZqrLCid6hU8okMh2keqFWQGqhlbMWr/kWA6C2HxMt8DRU0FISk2fmSvv0dKvCFGCSsQctKK4Df X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Jul 2026 14:00:48.6916 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 6941e974-5d29-43a5-9c94-08dee0e7247d X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d; Ip=[165.204.84.17]; Helo=[satlexmb07.amd.com] X-MS-Exchange-CrossTenant-AuthSource: CH1PEPF0000AD7C.namprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR12MB7254 X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces~public-inbox=simark.ca@sourceware.org This is yet another refactoring step to treat Harvard address space ids and address class ids separately and transparently from the fact that they are stored in type instance flags. The function 'address_space_name_to_type_instance_flags' converts address space and address class names to type instance flags. It deals with the Harvard address space names "code" and "data" as well as architecture specific address class names. As a result, it may produce type instance flags where either the Harvard address space bits or the address class bits are set. The function 'address_space_type_instance_flags_to_name' does the conversion in the opposite direction. Inline the functions and remove them. This is a step towards separating the handling of two concepts. In type-stack.c, which is used by the parser to convert user inputs into types, "@code" and "@data" and architecture-specific address class names are treated the same, too, blurring the difference and their storage in type instance flags. While we inline the use of address_space_name_to_type_instance_flags there, we also separate the two topics by defining different tokens. The patch still pushes type instance flags into the type stack. The subsequent patch will further clean this up to store address class and address space ids. A hardcoded shift operation will go away in a future patch. --- gdb/c-typeprint.c | 21 ++++++++-- gdb/gdbtypes.c | 46 ---------------------- gdb/gdbtypes.h | 6 --- gdb/testsuite/gdb.base/address_space_qualifier.exp | 4 +- gdb/type-stack.c | 37 +++++++++++++++-- gdb/type-stack.h | 31 +++++++++------ 6 files changed, 71 insertions(+), 74 deletions(-) diff --git a/gdb/c-typeprint.c b/gdb/c-typeprint.c index aaace85aa29..e3096a7859b 100644 --- a/gdb/c-typeprint.c +++ b/gdb/c-typeprint.c @@ -19,6 +19,7 @@ #include "event-top.h" #include "bfd.h" #include "symtab.h" +#include "gdbarch.h" #include "gdbtypes.h" #include "expression.h" #include "value.h" @@ -482,10 +483,22 @@ c_type_print_modifier (struct type *type, struct ui_file *stream, did_print_modifier = 1; } - address_space_id - = address_space_type_instance_flags_to_name (type->arch (), - type->instance_flags ()); - if (address_space_id) + address_space_id = nullptr; + + if (TYPE_CODE_SPACE (type)) + address_space_id = "code"; + else if (TYPE_DATA_SPACE (type)) + address_space_id = "data"; + else + { + unsigned int aclass = TYPE_ADDRESS_CLASS (type); + if (aclass != 0 + && gdbarch_address_class_id_to_name_p (type->arch ())) + address_space_id = gdbarch_address_class_id_to_name (type->arch (), + aclass); + } + + if (address_space_id != nullptr) { if (did_print_modifier || need_pre_space) gdb_printf (stream, " "); diff --git a/gdb/gdbtypes.c b/gdb/gdbtypes.c index f972a1f4278..97f1b3e1417 100644 --- a/gdb/gdbtypes.c +++ b/gdb/gdbtypes.c @@ -536,52 +536,6 @@ lookup_function_type_with_arguments (struct type *return_type, return create_function_type (return_type, nparams, param_types); } -/* Identify address space identifier by name -- return a - type_instance_flags. */ - -type_instance_flags -address_space_name_to_type_instance_flags (struct gdbarch *gdbarch, - const char *space_identifier) -{ - /* Check for known address space delimiters. */ - if (streq (space_identifier, "code")) - return TYPE_INSTANCE_FLAG_CODE_SPACE; - else if (streq (space_identifier, "data")) - return TYPE_INSTANCE_FLAG_DATA_SPACE; - - unsigned int aclass; - if (gdbarch_address_class_name_to_id_p (gdbarch) - && gdbarch_address_class_name_to_id (gdbarch, - space_identifier, - aclass)) - { - return (enum type_instance_flag_value) (aclass << 4); - } - else - error (_("Unknown address space specifier: \"%s\""), space_identifier); -} - -/* Identify address space identifier by type_instance_flags and return - the string version of the address space name. */ - -const char * -address_space_type_instance_flags_to_name (struct gdbarch *gdbarch, - type_instance_flags space_flag) -{ - if (space_flag & TYPE_INSTANCE_FLAG_CODE_SPACE) - return "code"; - else if (space_flag & TYPE_INSTANCE_FLAG_DATA_SPACE) - return "data"; - - unsigned int aclass = TYPE_ADDRESS_CLASS_FROM_INSTANCE_FLAGS (space_flag); - - if (aclass != 0 - && gdbarch_address_class_id_to_name_p (gdbarch)) - return gdbarch_address_class_id_to_name (gdbarch, aclass); - else - return NULL; -} - /* Create a new type with instance flags NEW_FLAGS, based on TYPE. If STORAGE is non-NULL, create the new type instance there. diff --git a/gdb/gdbtypes.h b/gdb/gdbtypes.h index 3c21d4ad214..1d5c96ab2c1 100644 --- a/gdb/gdbtypes.h +++ b/gdb/gdbtypes.h @@ -2420,12 +2420,6 @@ extern struct type *make_atomic_type (struct type *); extern void replace_type (struct type *, struct type *); -extern type_instance_flags address_space_name_to_type_instance_flags - (struct gdbarch *, const char *); - -extern const char *address_space_type_instance_flags_to_name - (struct gdbarch *, type_instance_flags); - extern struct type *make_type_with_address_space (struct type *type, type_instance_flags space_identifier); diff --git a/gdb/testsuite/gdb.base/address_space_qualifier.exp b/gdb/testsuite/gdb.base/address_space_qualifier.exp index 49437dad60d..3e94e895493 100644 --- a/gdb/testsuite/gdb.base/address_space_qualifier.exp +++ b/gdb/testsuite/gdb.base/address_space_qualifier.exp @@ -22,12 +22,12 @@ gdb_test_no_output "set language c" with_test_prefix "C" { gdb_test "p *(@somerandomqualifiername int *) 0x12345678" \ - "Unknown address space specifier: \"somerandomqualifiername\"" + "Unknown address space/class specifier: \"somerandomqualifiername\"" } gdb_test_no_output "set language c++" with_test_prefix "C++" { gdb_test "p *(@somerandomqualifiername int *) 0x12345678" \ - "Unknown address space specifier: \"somerandomqualifiername\"" + "Unknown address space/class specifier: \"somerandomqualifiername\"" } diff --git a/gdb/type-stack.c b/gdb/type-stack.c index 7e790a003ec..e2783eaab11 100644 --- a/gdb/type-stack.c +++ b/gdb/type-stack.c @@ -20,6 +20,7 @@ #include "type-stack.h" #include "gdbtypes.h" +#include "gdbarch.h" /* See type-stack.h. */ @@ -63,10 +64,35 @@ type_stack::insert (struct gdbarch *gdbarch, const char *string) else slot = 0; - element.piece = tp_space_identifier; + enum type_pieces piece; + int int_val; + unsigned int aclass; + + /* Check for Harvard address space delimiters. */ + if (streq (string, "code")) + { + piece = tp_harvard_aspace_identifier; + int_val = TYPE_INSTANCE_FLAG_CODE_SPACE; + } + else if (streq (string, "data")) + { + piece = tp_harvard_aspace_identifier; + int_val = TYPE_INSTANCE_FLAG_DATA_SPACE; + } + else if (gdbarch_address_class_name_to_id_p (gdbarch) + && gdbarch_address_class_name_to_id (gdbarch, + string, + aclass)) + { + piece = tp_aclass_identifier; + int_val = (enum type_instance_flag_value) (aclass << 4); + } + else + error (_("Unknown address space/class specifier: \"%s\""), string); + + element.piece = piece; insert_into (slot, element); - element.int_val - = address_space_name_to_type_instance_flags (gdbarch, string); + element.int_val = int_val; insert_into (slot, element); } @@ -125,7 +151,10 @@ type_stack::follow_types (struct type *follow_type) case tp_volatile: make_volatile = 1; break; - case tp_space_identifier: + case tp_harvard_aspace_identifier: + make_addr_space = (enum type_instance_flag_value) pop_int (); + break; + case tp_aclass_identifier: make_addr_space = (enum type_instance_flag_value) pop_int (); break; case tp_atomic: diff --git a/gdb/type-stack.h b/gdb/type-stack.h index 318ef715b2e..85e07549ee9 100644 --- a/gdb/type-stack.h +++ b/gdb/type-stack.h @@ -45,9 +45,12 @@ enum type_pieces tp_function_with_arguments, tp_const, tp_volatile, - /* An address space identifier. The address space is also pushed on + /* An Harvard address space identifier (i.e. "code" or "data"). The + address space is also pushed on the stack. */ + tp_harvard_aspace_identifier, + /* An address class identifier. The address class is also pushed on the stack. */ - tp_space_identifier, + tp_aclass_identifier, tp_atomic, tp_restrict, /* A separate type stack, which is also pushed onto this type @@ -114,7 +117,8 @@ struct type_stack accept an integer argument are allowed. */ void push (enum type_pieces tp, int n) { - gdb_assert (tp == tp_array || tp == tp_space_identifier || tp == tp_kind); + gdb_assert (tp == tp_array || tp == tp_harvard_aspace_identifier + || tp == tp_aclass_identifier || tp == tp_kind); type_stack_elt elt; elt.int_val = n; m_elements.push_back (elt); @@ -172,7 +176,8 @@ struct type_stack type_stack_elt elt = m_elements.back (); m_elements.pop_back (); type_pieces tp = elt.piece; - gdb_assert (tp == tp_array || tp == tp_space_identifier || tp == tp_kind); + gdb_assert (tp == tp_array || tp == tp_harvard_aspace_identifier + || tp == tp_aclass_identifier || tp == tp_kind); elt = m_elements.back (); m_elements.pop_back (); return elt.int_val; @@ -204,13 +209,14 @@ struct type_stack return elt.stack_val; } - /* Insert a tp_space_identifier and the corresponding address space - value into the stack. STRING is the name of an address space, as - recognized by address_space_name_to_type_instance_flags. If the - stack is empty, the new elements are simply pushed. If the stack - is not empty, this function assumes that the first item on the - stack is a tp_pointer, and the new values are inserted above the - first item. */ + /* Insert an address space or address class identifier and the + corresponding id value into the stack. STRING is the name of a + Harvard address space ("code" or "data"), or the name of an + address class as recognized by gdbarch_address_class_name_to_id. + If the stack is empty, the new elements are simply pushed. If + the stack is not empty, this function assumes that the first item + on the stack is a tp_pointer, and the new values are inserted + above the first item. */ void insert (struct gdbarch *gdbarch, const char *string); @@ -252,7 +258,8 @@ struct type_stack { return (tp == tp_array || tp == tp_kind || tp == tp_type_stack || tp == tp_function_with_arguments - || tp == tp_space_identifier); + || tp == tp_harvard_aspace_identifier + || tp == tp_aclass_identifier); } -- 2.34.1