This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Remove a redundant statement in predict.c
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Yi Yang <ahyangyi at google dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 8 Aug 2014 10:53:20 +0200
- Subject: Re: [PATCH] Remove a redundant statement in predict.c
- Authentication-results: sourceware.org; auth=none
- References: <CADvRseaHvYEZd7JMrXO+dNqAg+-z5Pq1hjSJtHfALUsXeKY36w at mail dot gmail dot com>
On Fri, Aug 8, 2014 at 1:59 AM, Yi Yang <ahyangyi@google.com> wrote:
> Remove a redundant assignment "*predictor = PRED_BUILTIN_EXPECT;",
> since six lines later *predictor is assigned again.
Ok.
Thanks,
Richard.
> --
>
> 2014-08-07 Yi Yang <ahyangyi@google.com>
>
> gcc:
> * predict.c (expr_expected_value_1): Remove the redundant assignment.
>
> diff --git gcc/predict.c gcc/predict.c
> index 835c618..869fc5d 100644
> --- gcc/predict.c
> +++ gcc/predict.c
> @@ -1858,7 +1858,6 @@ expr_expected_value_1 (tree type, tree op0, enum
> tree_code code,
> return val;
> if (predictor)
> {
> - *predictor = PRED_BUILTIN_EXPECT;
> tree val2 = gimple_call_arg (def, 2);
> gcc_assert (TREE_CODE (val2) == INTEGER_CST
> && tree_fits_uhwi_p (val2)