This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH][GCC][mid-end] Fix single use requirement for xorsign (x * copysign (1, y))


Sorry, forgot to add some maintainers :)
________________________________________
From: gcc-patches-owner@gcc.gnu.org <gcc-patches-owner@gcc.gnu.org> on behalf of Tamar Christina <tamar.christina@arm.com>
Sent: Tuesday, August 22, 2017 5:27:23 PM
To: gcc-patches@gcc.gnu.org
Cc: nd; law@redhat.com
Subject: [PATCH][GCC][mid-end] Fix single use requirement for xorsign (x * copysign (1, y))

Hi All,

This patch fixes the placement of the single-use requirement for xorsign.

on `x = a * copysign (1, b)` the intention was that `copysign (1, b)` be
single use, and not `x`. Requiring `x` to be single use blocks transformation
where we do want it to occur.

Regtested on aarch64-none-linux-gnu and no regressions
(only target currently supporting this).

Ok for trunk?

gcc/
2017-08-22  Tamar Christina  <tamar.christina@arm.com>

        PR middle-end/19706
        * tree-ssa-math-opts.c (convert_expand_mult_copysign):
        Fix single-use check.

--


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]