This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH GCC][v2]Simplify alias check code generation in vectorizer
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Bernd Edlinger <bernd dot edlinger at hotmail dot de>,Robin Dapp <rdapp at linux dot vnet dot ibm dot com>
- Cc: Markus Trippelsdorf <markus at trippelsdorf dot de>,Richard Biener <rguenther at suse dot de>,GCC Patches <gcc-patches at gcc dot gnu dot org>,"Bin.Cheng" <amker dot cheng at gmail dot com>
- Date: Mon, 26 Sep 2016 20:15:54 +0200
- Subject: Re: [PATCH GCC][v2]Simplify alias check code generation in vectorizer
- Authentication-results: sourceware.org; auth=none
- References: <AM4PR0701MB2162A6D6AFF36191EF65F1A2E4CD0@AM4PR0701MB2162.eurprd07.prod.outlook.com>
On September 26, 2016 5:46:28 PM GMT+02:00, Bernd Edlinger <bernd.edlinger@hotmail.de> wrote:
>Hi,
>
>>@@ -2310,7 +2313,8 @@ create_intersect_range_checks_index
>(loop_vec_info loop_vinfo, tree *cond_expr,
>> gcc_assert (TREE_CODE (DR_STEP (dr_a.dr)) == INTEGER_CST);
>>
>> bool neg_step = tree_int_cst_compare (DR_STEP (dr_a.dr),
>size_zero_node) < 0;
>>- unsigned HOST_WIDE_INT abs_step = tree_to_uhwi (DR_STEP (dr_a.dr));
>>+ unsigned HOST_WIDE_INT abs_step =
>>+ absu_hwi (tree_to_shwi (DR_STEP (dr_a.dr)));
>> if (neg_step)
>> abs_step = -abs_step;
>
>Hmmm...
>
>but you must remove the neg_step if you do this.
The negation, yes. Neg_step is used later as well.
Richard.
>Right?
>
>
>Bernd.