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] PR target/66824: Allow software FP SFmode in FP splitter

From: Uros Bizjak <ubizjak at gmail dot com>
To: "H.J. Lu" <hjl dot tools at gmail dot com>
Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Thu, 16 Jul 2015 09:41:28 +0200
Subject: Re: [PATCH] PR target/66824: Allow software FP SFmode in FP splitter
Authentication-results: sourceware.org; auth=none
References: <20150709191338 dot GA23448 at intel dot com> <20150709205822 dot GA21880 at intel dot com> <20150711192325 dot GA2214 at gmail dot com> <CAFULd4b3qnTHmwJ5_Om6HEcJLCt+VChtq8PioCXb=eTcvVRorw at mail dot gmail dot com> <CAMe9rOqdOAYUPwo9soKZSGx3Xwqt5SfDVFSm_THoLzSdnNrzgw at mail dot gmail dot com> <CAFULd4bsZn_zDB6j=yS9kSSj2_NaVTCiqy0-MaurREBS91HLqQ at mail dot gmail dot com> <CAFULd4ZHb7_M_+2Cxomabmf0UwhcTnbzK+H16qhSwoKDAwgWMA at mail dot gmail dot com> <CAMe9rOp8=U2zBDbwF2pBH7aHtoUOzbsKpCv_wL1OPNb33ZW9Rg at mail dot gmail dot com>

On Wed, Jul 15, 2015 at 3:37 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Wed, Jul 15, 2015 at 5:05 AM, Uros Bizjak <ubizjak@gmail.com> wrote:
>> On Wed, Jul 15, 2015 at 1:36 PM, Uros Bizjak <ubizjak@gmail.com> wrote:
>>
>>>> I couldn't figure a way to add conditional constraints for "?r/rmF" and
>>>> "r/rmF".   I simply disabled *movsf_internal if TARGET_HARD_FP_REGS
>>>> is false and added a new "*movsf_internal_soft_fp" pattern.
>>>>
>>>> OK for trunk if there is no regressions?
>>>
>>> No. We don't want duplicated patterns.
>>> Please add two patterns at the end of the constraint string and use
>>
>> ... add two *constraints* at the end ...
>
> When I simply added "r/rmF" to the end,  RA saw "?r/rmF" first and
> ignored  "r/rmF" at the end.

Please try attached patch.

Uros.

Attachment: p.diff.txt
Description: Text document

References:
- [PATCH] PR target/66824: -miamcu doesn't load FP constant into register directly
  - From: H.J. Lu
- Re: [PATCH] PR target/66824: -miamcu doesn't load FP constant into register directly
  - From: H.J. Lu
- [PATCH] PR target/66824: Allow software FP SFmode in FP splitter
  - From: H.J. Lu
- Re: [PATCH] PR target/66824: Allow software FP SFmode in FP splitter
  - From: Uros Bizjak
- Re: [PATCH] PR target/66824: Allow software FP SFmode in FP splitter
  - From: H.J. Lu
- Re: [PATCH] PR target/66824: Allow software FP SFmode in FP splitter
  - From: Uros Bizjak
- Re: [PATCH] PR target/66824: Allow software FP SFmode in FP splitter
  - From: Uros Bizjak
- Re: [PATCH] PR target/66824: Allow software FP SFmode in FP splitter
  - From: H.J. Lu

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