RE: [PATCH][GCC][ARM] Fix can_change_mode_class for big-endian

2018-07-06 Thread Tamar Christina
Hi Christoph, > 90d62e699bce9594879be2e3016c9b36c7e064c8..703632240822e762a90657096 > > >> 4b949c783df56f3 100644 > > >> --- a/gcc/config/arm/arm.c > > >> +++ b/gcc/config/arm/arm.c > > >> @@ -31508,8 +31508,8 @@ arm_can_change_mode_class > (machine_mode from, machine_mode to, > > >>{ > > >>

Re: [PATCH][GCC][ARM] Fix can_change_mode_class for big-endian

2018-07-06 Thread Christophe Lyon
Hi Tamar, On Wed, 20 Jun 2018 at 15:35, Kyrill Tkachov wrote: > > > On 20/06/18 11:33, Tamar Christina wrote: > > Hi Kyrill, > > > > Many thanks for the review! > > > > The 06/20/2018 09:43, Kyrill Tkachov wrote: > >> Hi Tamar, > >> > >> On 19/06/18 15:14, Tamar Christina wrote: > >>> Hi All, > >

Re: [PATCH][GCC][ARM] Fix can_change_mode_class for big-endian

2018-06-20 Thread Kyrill Tkachov
On 20/06/18 11:33, Tamar Christina wrote: Hi Kyrill, Many thanks for the review! The 06/20/2018 09:43, Kyrill Tkachov wrote: Hi Tamar, On 19/06/18 15:14, Tamar Christina wrote: Hi All, This patch requires https://gcc.gnu.org/ml/gcc-patches/2018-06/msg01145.html to work, it has been accep

Re: [PATCH][GCC][ARM] Fix can_change_mode_class for big-endian

2018-06-20 Thread Tamar Christina
Hi Kyrill, Many thanks for the review! The 06/20/2018 09:43, Kyrill Tkachov wrote: > Hi Tamar, > > On 19/06/18 15:14, Tamar Christina wrote: > > Hi All, > > > > This patch requires > > https://gcc.gnu.org/ml/gcc-patches/2018-06/msg01145.html to work, > > it has been accepted once already but ca

Re: [PATCH][GCC][ARM] Fix can_change_mode_class for big-endian

2018-06-20 Thread Kyrill Tkachov
Hi Tamar, On 19/06/18 15:14, Tamar Christina wrote: Hi All, This patch requires https://gcc.gnu.org/ml/gcc-patches/2018-06/msg01145.html to work, it has been accepted once already but caused a regression on certain configuratoins. I am re-submitting it with the required mid-end change and req

Re: [PATCH][GCC][ARM] Fix can_change_mode_class for big-endian

2018-03-19 Thread Tamar Christina
Sending to list Thu 03/19/2018 08:48, Tamar Christina wrote: > Hi Christophe, > > The 03/16/2018 13:50, Christophe Lyon wrote: > > On 15 March 2018 at 11:19, Kyrill Tkachov > > wrote: > > > Hi Tamar, > > > > > > > > >> Regtested on armeb-none-eabi and no regressions. > > >> Bootstrapped on arm

Re: [PATCH][GCC][ARM] Fix can_change_mode_class for big-endian

2018-03-16 Thread Christophe Lyon
On 15 March 2018 at 11:19, Kyrill Tkachov wrote: > Hi Tamar, > > > On 05/03/18 16:51, Tamar Christina wrote: >> >> Hi All, >> >> Taking the subreg of a vector mode on big-endian may result in an infinite >> recursion and eventually a segfault once we run out of stack space. >> >> As an example, t

Re: [PATCH][GCC][ARM] Fix can_change_mode_class for big-endian

2018-03-15 Thread Kyrill Tkachov
Hi Tamar, On 05/03/18 16:51, Tamar Christina wrote: Hi All, Taking the subreg of a vector mode on big-endian may result in an infinite recursion and eventually a segfault once we run out of stack space. As an example, taking a subreg of V4HF to SImode we end up in the following loop on big-end

RE: [PATCH][GCC][ARM] Fix can_change_mode_class for big-endian

2018-03-12 Thread Tamar Christina
Ping > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Tamar Christina > Sent: Monday, March 5, 2018 16:52 > To: gcc-patches@gcc.gnu.org > Cc: nd ; Ramana Radhakrishnan > ; Richard Earnshaw > ; ni...@redhat.com; Kyrylo Tkachov