SCCVN: Fix repeating variable name "len"
Checks
Commit Message
From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>
Line 3292: has variable name "len": tree mask = NULL_TREE, len = NULL_TREE, bias = NULL_TREE;
Line 3349: has variable name "len": HOST_WIDE_INT start = 0, len = 0;
Since they are never used simultaneously, such issue is not recognized for now.
However, I want to add LEN_MASK_{LOAD,STORE} which will need these 2 variables, so fix naming in this path.
Change HOST_WIDE_INT start = 0, len = 0; into HOST_WIDE_INT start = 0, length = 0;
gcc/ChangeLog:
* tree-ssa-sccvn.cc (vn_reference_lookup_3): Change name "len" into "length".
---
gcc/tree-ssa-sccvn.cc | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
Comments
On Mon, 26 Jun 2023, juzhe.zhong@rivai.ai wrote:
> From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>
>
> Line 3292: has variable name "len": tree mask = NULL_TREE, len = NULL_TREE, bias = NULL_TREE;
> Line 3349: has variable name "len": HOST_WIDE_INT start = 0, len = 0;
>
> Since they are never used simultaneously, such issue is not recognized for now.
> However, I want to add LEN_MASK_{LOAD,STORE} which will need these 2 variables, so fix naming in this path.
>
> Change HOST_WIDE_INT start = 0, len = 0; into HOST_WIDE_INT start = 0, length = 0;
OK.
> gcc/ChangeLog:
>
> * tree-ssa-sccvn.cc (vn_reference_lookup_3): Change name "len" into "length".
>
> ---
> gcc/tree-ssa-sccvn.cc | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/gcc/tree-ssa-sccvn.cc b/gcc/tree-ssa-sccvn.cc
> index 27c84e78fcf..11061a374a2 100644
> --- a/gcc/tree-ssa-sccvn.cc
> +++ b/gcc/tree-ssa-sccvn.cc
> @@ -3346,17 +3346,17 @@ vn_reference_lookup_3 (ao_ref *ref, tree vuse, void *data_,
> = tree_to_uhwi (TYPE_SIZE (TREE_TYPE (vectype)));
> if (mask)
> {
> - HOST_WIDE_INT start = 0, len = 0;
> + HOST_WIDE_INT start = 0, length = 0;
> unsigned mask_idx = 0;
> do
> {
> if (integer_zerop (VECTOR_CST_ELT (mask, mask_idx)))
> {
> - if (len != 0)
> + if (length != 0)
> {
> pd.rhs_off = start;
> pd.offset = offset2i + start;
> - pd.size = len;
> + pd.size = length;
> if (ranges_known_overlap_p
> (offset, maxsize, pd.offset, pd.size))
> {
> @@ -3367,18 +3367,18 @@ vn_reference_lookup_3 (ao_ref *ref, tree vuse, void *data_,
> }
> }
> start = (mask_idx + 1) * elsz;
> - len = 0;
> + length = 0;
> }
> else
> - len += elsz;
> + length += elsz;
> mask_idx++;
> }
> while (known_lt (mask_idx, TYPE_VECTOR_SUBPARTS (vectype)));
> - if (len != 0)
> + if (length != 0)
> {
> pd.rhs_off = start;
> pd.offset = offset2i + start;
> - pd.size = len;
> + pd.size = length;
> if (ranges_known_overlap_p (offset, maxsize,
> pd.offset, pd.size))
> return data->push_partial_def (pd, set, set,
>
Committed, thanks Richard.
Pan
-----Original Message-----
From: Gcc-patches <gcc-patches-bounces+pan2.li=intel.com@gcc.gnu.org> On Behalf Of Richard Biener via Gcc-patches
Sent: Monday, June 26, 2023 3:53 PM
To: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>
Cc: gcc-patches@gcc.gnu.org; richard.sandiford@arm.com
Subject: Re: [PATCH] SCCVN: Fix repeating variable name "len"
On Mon, 26 Jun 2023, juzhe.zhong@rivai.ai wrote:
> From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>
>
> Line 3292: has variable name "len": tree mask = NULL_TREE, len = NULL_TREE, bias = NULL_TREE;
> Line 3349: has variable name "len": HOST_WIDE_INT start = 0, len = 0;
>
> Since they are never used simultaneously, such issue is not recognized for now.
> However, I want to add LEN_MASK_{LOAD,STORE} which will need these 2 variables, so fix naming in this path.
>
> Change HOST_WIDE_INT start = 0, len = 0; into HOST_WIDE_INT start = 0, length = 0;
OK.
> gcc/ChangeLog:
>
> * tree-ssa-sccvn.cc (vn_reference_lookup_3): Change name "len" into "length".
>
> ---
> gcc/tree-ssa-sccvn.cc | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/gcc/tree-ssa-sccvn.cc b/gcc/tree-ssa-sccvn.cc
> index 27c84e78fcf..11061a374a2 100644
> --- a/gcc/tree-ssa-sccvn.cc
> +++ b/gcc/tree-ssa-sccvn.cc
> @@ -3346,17 +3346,17 @@ vn_reference_lookup_3 (ao_ref *ref, tree vuse, void *data_,
> = tree_to_uhwi (TYPE_SIZE (TREE_TYPE (vectype)));
> if (mask)
> {
> - HOST_WIDE_INT start = 0, len = 0;
> + HOST_WIDE_INT start = 0, length = 0;
> unsigned mask_idx = 0;
> do
> {
> if (integer_zerop (VECTOR_CST_ELT (mask, mask_idx)))
> {
> - if (len != 0)
> + if (length != 0)
> {
> pd.rhs_off = start;
> pd.offset = offset2i + start;
> - pd.size = len;
> + pd.size = length;
> if (ranges_known_overlap_p
> (offset, maxsize, pd.offset, pd.size))
> {
> @@ -3367,18 +3367,18 @@ vn_reference_lookup_3 (ao_ref *ref, tree vuse, void *data_,
> }
> }
> start = (mask_idx + 1) * elsz;
> - len = 0;
> + length = 0;
> }
> else
> - len += elsz;
> + length += elsz;
> mask_idx++;
> }
> while (known_lt (mask_idx, TYPE_VECTOR_SUBPARTS (vectype)));
> - if (len != 0)
> + if (length != 0)
> {
> pd.rhs_off = start;
> pd.offset = offset2i + start;
> - pd.size = len;
> + pd.size = length;
> if (ranges_known_overlap_p (offset, maxsize,
> pd.offset, pd.size))
> return data->push_partial_def (pd, set, set,
>
@@ -3346,17 +3346,17 @@ vn_reference_lookup_3 (ao_ref *ref, tree vuse, void *data_,
= tree_to_uhwi (TYPE_SIZE (TREE_TYPE (vectype)));
if (mask)
{
- HOST_WIDE_INT start = 0, len = 0;
+ HOST_WIDE_INT start = 0, length = 0;
unsigned mask_idx = 0;
do
{
if (integer_zerop (VECTOR_CST_ELT (mask, mask_idx)))
{
- if (len != 0)
+ if (length != 0)
{
pd.rhs_off = start;
pd.offset = offset2i + start;
- pd.size = len;
+ pd.size = length;
if (ranges_known_overlap_p
(offset, maxsize, pd.offset, pd.size))
{
@@ -3367,18 +3367,18 @@ vn_reference_lookup_3 (ao_ref *ref, tree vuse, void *data_,
}
}
start = (mask_idx + 1) * elsz;
- len = 0;
+ length = 0;
}
else
- len += elsz;
+ length += elsz;
mask_idx++;
}
while (known_lt (mask_idx, TYPE_VECTOR_SUBPARTS (vectype)));
- if (len != 0)
+ if (length != 0)
{
pd.rhs_off = start;
pd.offset = offset2i + start;
- pd.size = len;
+ pd.size = length;
if (ranges_known_overlap_p (offset, maxsize,
pd.offset, pd.size))
return data->push_partial_def (pd, set, set,