From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ej1-x62e.google.com (mail-ej1-x62e.google.com [IPv6:2a00:1450:4864:20::62e]) by sourceware.org (Postfix) with ESMTPS id 466A2385841C; Sat, 13 May 2023 23:23:54 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 466A2385841C Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-ej1-x62e.google.com with SMTP id a640c23a62f3a-965c3f9af2aso1728227666b.0; Sat, 13 May 2023 16:23:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1684020232; x=1686612232; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=UwdphHobJdHdr7kpfathxNdrPsZPCvK03KU6b/A3WQE=; b=eg47OTsKmozH+RexbkBhWYCtJCa5fRWUg2OwdPP5hM7lNXpZWNP4ua2NZNHlclFmTu VtVPHdTYN4ip6P9rQlGcTqr9c39QyRhY64ePkyK89zqBOefjFsDjtz+mrRrJnfuidGpB ZdM7SbDvweLzkFzCfSfINJXxqvEPayaNbBnmuc/DakOOLDi0jCqICqBNXOaPjIxqmpiH Axkve4fwgVrR4UdkzKxXOOyuLYrKrKSpQ+okUJmGNvQlTpDUQ7QXSp2lbPEoY0DG+Ri2 muAMoo1n14HJj03Qr6LLgyJptXzd6oVKLYP5U/CuzVNQ+ds/74F7Fj6++Km4Xc2U0VMH uf3w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684020232; x=1686612232; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=UwdphHobJdHdr7kpfathxNdrPsZPCvK03KU6b/A3WQE=; b=IRfNdYaGxUaqJDtJxlmvKYFI5XUNtdw5jjyheIqDhgSxaQqY89+7L/NE6E9lBhEmuL a1ViE3WAk2xGhWUMzSCbgTZpZDzjGfI2JciHQ9XJveq1VcX1COBrZdUYiv+v3cQ6morz T4TCnCmb6foX0cmU1VHWYhIeIkZAr+CUvjPmFm9iCluWDcs75caSDZG+Q9if3Bt1Shlm ofKze37jUgEYAbdzzcMfETXBhV8oIs0sJgkIgd1+XTX2KFh3pmVt7S8uJ22e5t6dji9C nZ1uTmUKAFEKqI3b8vkMYQQt1HzyGBSHEmw9o46TuxGim7ex3GoxRFIkN3o86vtR2vEQ 2RuA== X-Gm-Message-State: AC+VfDwtzw/nMM12vPS5nQHp2pV5BFbHkqDf0eyzv73YAnlx/DD6wVS8 eBWm4xDE1Z6hocsMoSScJ3JTw69Morw= X-Google-Smtp-Source: ACHHUZ6D9y1rcE9v8B90/A3YVLU8tf/OFZ8sCBDsnqek7eCu6xKX0BnGEnr7h7lxdzewJ1hmn5whUg== X-Received: by 2002:a17:907:7fa4:b0:96a:8ef4:e1bd with SMTP id qk36-20020a1709077fa400b0096a8ef4e1bdmr8446047ejc.18.1684020232035; Sat, 13 May 2023 16:23:52 -0700 (PDT) Received: from nbbrfq ([2001:871:227:ce8b:afc3:c345:e1dd:564e]) by smtp.gmail.com with ESMTPSA id hy27-20020a1709068a7b00b0094f1fe1696bsm7260438ejc.216.2023.05.13.16.23.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 13 May 2023 16:23:48 -0700 (PDT) Received: from b by nbbrfq with local (Exim 4.96) (envelope-from ) id 1pxyaa-001Amn-0R; Sun, 14 May 2023 01:23:48 +0200 From: Bernhard Reutner-Fischer To: gcc-patches@gcc.gnu.org Cc: Bernhard Reutner-Fischer Subject: [PATCH 09/14] rust: use _P() defines from tree.h Date: Sun, 14 May 2023 01:23:16 +0200 Message-Id: <20230513232321.279733-10-rep.dot.nop@gmail.com> X-Mailer: git-send-email 2.40.1 In-Reply-To: <20230513232321.279733-1-rep.dot.nop@gmail.com> References: <20230513232321.279733-1-rep.dot.nop@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-10.3 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM,GIT_PATCH_0,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: From: Bernhard Reutner-Fischer gcc/rust/ChangeLog: * backend/rust-compile-expr.cc (CompileExpr::type_cast_expression): Use _P() defines from tree.h * backend/rust-tree.cc (build_cplus_array_type): Ditto. * backend/rust-tree.h (ARITHMETIC_TYPE_P): Ditto. (gnu_vector_type_p): Ditto. * checks/lints/rust-lint-unused-var.cc (check_decl): Ditto. * rust-gcc.cc (Gcc_backend::fill_in_array): Ditto. (Gcc_backend::named_type): Ditto. (Gcc_backend::convert_expression): Ditto. (Gcc_backend::init_statement): Ditto. --- gcc/rust/backend/rust-compile-expr.cc | 2 +- gcc/rust/backend/rust-tree.cc | 2 +- gcc/rust/backend/rust-tree.h | 4 ++-- gcc/rust/checks/lints/rust-lint-unused-var.cc | 2 +- gcc/rust/rust-gcc.cc | 8 ++++---- 5 files changed, 9 insertions(+), 9 deletions(-) diff --git a/gcc/rust/backend/rust-compile-expr.cc b/gcc/rust/backend/rust-compile-expr.cc index d7945dbf26b..3dc34828e32 100644 --- a/gcc/rust/backend/rust-compile-expr.cc +++ b/gcc/rust/backend/rust-compile-expr.cc @@ -2267,7 +2267,7 @@ CompileExpr::type_cast_expression (tree type_to_cast_to, tree expr_tree, // FIXME check for TREE_OVERFLOW? return cast; } - else if (TREE_CODE (type_to_cast_to) == REAL_TYPE) + else if (SCALAR_FLOAT_TYPE_P (type_to_cast_to)) { tree cast = convert_to_real (type_to_cast_to, expr_tree); // FIXME diff --git a/gcc/rust/backend/rust-tree.cc b/gcc/rust/backend/rust-tree.cc index 8243d4cf5c6..7e11e6584ae 100644 --- a/gcc/rust/backend/rust-tree.cc +++ b/gcc/rust/backend/rust-tree.cc @@ -2404,7 +2404,7 @@ build_cplus_array_type (tree elt_type, tree index_type, int dependent) } /* Avoid spurious warnings with VLAs (c++/54583). */ - if (TYPE_SIZE (t) && EXPR_P (TYPE_SIZE (t))) + if (CAN_HAVE_LOCATION_P (TYPE_SIZE (t))) suppress_warning (TYPE_SIZE (t), OPT_Wunused); /* Push these needs up to the ARRAY_TYPE so that initialization takes diff --git a/gcc/rust/backend/rust-tree.h b/gcc/rust/backend/rust-tree.h index 284fd873c1c..6d243b7f86e 100644 --- a/gcc/rust/backend/rust-tree.h +++ b/gcc/rust/backend/rust-tree.h @@ -54,7 +54,7 @@ Keep these checks in ascending code order. */ #define ARITHMETIC_TYPE_P(TYPE) \ - (RS_INTEGRAL_TYPE_P (TYPE) || TREE_CODE (TYPE) == REAL_TYPE \ + (RS_INTEGRAL_TYPE_P (TYPE) || SCALAR_FLOAT_TYPE_P (TYPE) \ || TREE_CODE (TYPE) == COMPLEX_TYPE) /* True iff TYPE is cv decltype(nullptr). */ @@ -3250,7 +3250,7 @@ identifier_p (tree t) inline bool gnu_vector_type_p (const_tree type) { - return TREE_CODE (type) == VECTOR_TYPE && !TYPE_INDIVISIBLE_P (type); + return VECTOR_TYPE_P (type) && !TYPE_INDIVISIBLE_P (type); } extern vec * diff --git a/gcc/rust/checks/lints/rust-lint-unused-var.cc b/gcc/rust/checks/lints/rust-lint-unused-var.cc index ba5ffb9372b..2cf5cd60f15 100644 --- a/gcc/rust/checks/lints/rust-lint-unused-var.cc +++ b/gcc/rust/checks/lints/rust-lint-unused-var.cc @@ -25,7 +25,7 @@ namespace Analysis { static void check_decl (tree *t) { - rust_assert (TREE_CODE (*t) == VAR_DECL || TREE_CODE (*t) == PARM_DECL + rust_assert (VAR_P (*t) || TREE_CODE (*t) == PARM_DECL || TREE_CODE (*t) == CONST_DECL); tree var_name = DECL_NAME (*t); diff --git a/gcc/rust/rust-gcc.cc b/gcc/rust/rust-gcc.cc index cf20b5b98e0..b1995bdb56a 100644 --- a/gcc/rust/rust-gcc.cc +++ b/gcc/rust/rust-gcc.cc @@ -886,7 +886,7 @@ Gcc_backend::fill_in_array (tree fill, tree element_type, tree length_tree) if (element_type == error_mark_node || length_tree == error_mark_node) return error_mark_node; - gcc_assert (TYPE_SIZE (element_type) != NULL_TREE); + gcc_assert (COMPLETE_TYPE_P (element_type)); length_tree = fold_convert (sizetype, length_tree); @@ -923,7 +923,7 @@ Gcc_backend::named_type (const std::string &name, tree type, Location location) // give it whatever Rust name we have at this point. if (TYPE_NAME (type) == NULL_TREE && location.gcc_location () == BUILTINS_LOCATION - && (TREE_CODE (type) == INTEGER_TYPE || TREE_CODE (type) == REAL_TYPE + && (TREE_CODE (type) == INTEGER_TYPE || SCALAR_FLOAT_TYPE_P (type) || TREE_CODE (type) == COMPLEX_TYPE || TREE_CODE (type) == BOOLEAN_TYPE)) { @@ -1173,7 +1173,7 @@ Gcc_backend::convert_expression (tree type_tree, tree expr_tree, } else if (TREE_CODE (type_tree) == INTEGER_TYPE) ret = convert_to_integer (type_tree, expr_tree); - else if (TREE_CODE (type_tree) == REAL_TYPE) + else if (SCALAR_FLOAT_TYPE_P (type_tree)) ret = convert_to_real (type_tree, expr_tree); else if (TREE_CODE (type_tree) == COMPLEX_TYPE) ret = convert_to_complex (type_tree, expr_tree); @@ -1935,7 +1935,7 @@ Gcc_backend::init_statement (tree, Bvariable *var, tree init_tree) tree var_tree = var->get_decl (); if (var_tree == error_mark_node || init_tree == error_mark_node) return error_mark_node; - gcc_assert (TREE_CODE (var_tree) == VAR_DECL); + gcc_assert (VAR_P (var_tree)); // To avoid problems with GNU ld, we don't make zero-sized // externally visible variables. That might lead us to doing an -- 2.30.2