On Fri, Dec 6, 2013 at 7:53 PM, Alan Modra wrote:
> On Fri, Dec 06, 2013 at 05:23:28PM -0500, Vladimir Makarov wrote:
>> On 12/6/2013, 2:40 PM, David Edelsohn wrote:
>> >On Fri, Dec 6, 2013 at 2:02 PM, Vladimir Makarov
>> >wrote:
>> >> * config/rs6000/rs600.md (*bswapdi2_64bit): Remove ?
On Fri, Dec 06, 2013 at 05:23:28PM -0500, Vladimir Makarov wrote:
> On 12/6/2013, 2:40 PM, David Edelsohn wrote:
> >On Fri, Dec 6, 2013 at 2:02 PM, Vladimir Makarov wrote:
> >> * config/rs6000/rs600.md (*bswapdi2_64bit): Remove ?? from the
> >> constraint.
> >
> >Okay, let's just r
On 12/6/2013, 2:40 PM, David Edelsohn wrote:
On Fri, Dec 6, 2013 at 2:02 PM, Vladimir Makarov wrote:
Here is the patch.
Tested and bootstrapped on gcc110.fsffrance.org.
Ok to commit?
2013-12-05 Vladimir Makarov
* config/rs6000/rs600.md (*bswapdi2_64bit): Remove ?? from the
On Fri, Dec 6, 2013 at 2:02 PM, Vladimir Makarov wrote:
> On 12/6/2013, 12:30 PM, Vladimir Makarov wrote:
>>
>> On 12/6/2013, 11:28 AM, Michael Meissner wrote:
>>>
>>> On Thu, Dec 05, 2013 at 12:40:17PM -0500, Vladimir Makarov wrote:
The following patch fixes two GCC testsuite failures f
On 12/6/2013, 12:30 PM, Vladimir Makarov wrote:
On 12/6/2013, 11:28 AM, Michael Meissner wrote:
On Thu, Dec 05, 2013 at 12:40:17PM -0500, Vladimir Makarov wrote:
The following patch fixes two GCC testsuite failures for LRA. The
patch makes swap through registers instead of memory for the test
On 12/6/2013, 11:28 AM, Michael Meissner wrote:
On Thu, Dec 05, 2013 at 12:40:17PM -0500, Vladimir Makarov wrote:
The following patch fixes two GCC testsuite failures for LRA. The
patch makes swap through registers instead of memory for the test
cases when LRA is used.
There are differences in
On Thu, Dec 05, 2013 at 12:40:17PM -0500, Vladimir Makarov wrote:
> The following patch fixes two GCC testsuite failures for LRA. The
> patch makes swap through registers instead of memory for the test
> cases when LRA is used.
>
> There are differences in reload and LRA constraint matching
> alg
On Fri, Dec 06, 2013 at 10:59:37AM -0500, Vladimir Makarov wrote:
> It is still two different patterns. One for reload and one for LRA.
> Attribute enabled is mostly used to describe insn constraints for
> subtargets.
I meant something like:
(define_insn "*bswapdi2_64bit"
[(set (match_operand:D
On 12/6/2013, 10:45 AM, Jakub Jelinek wrote:
On Fri, Dec 06, 2013 at 10:39:29AM -0500, Vladimir Makarov wrote:
Ok. I guess there is only one option to use one pattern for LRA
and reload without ?? in register alternative. In this case, reload
and LRA will actually work according to GCC docum
On Fri, Dec 06, 2013 at 10:39:29AM -0500, Vladimir Makarov wrote:
> Ok. I guess there is only one option to use one pattern for LRA
> and reload without ?? in register alternative. In this case, reload
> and LRA will actually work according to GCC documentation (LRA
> treats ? cost as the cost o
On 12/6/2013, 8:44 AM, David Edelsohn wrote:
On Thu, Dec 5, 2013 at 12:40 PM, Vladimir Makarov wrote:
The following patch fixes two GCC testsuite failures for LRA. The patch
makes swap through registers instead of memory for the test cases when LRA
is used.
There are differences in reload and
On Thu, Dec 5, 2013 at 12:40 PM, Vladimir Makarov wrote:
> The following patch fixes two GCC testsuite failures for LRA. The patch
> makes swap through registers instead of memory for the test cases when LRA
> is used.
>
> There are differences in reload and LRA constraint matching algorithm whic
The following patch fixes two GCC testsuite failures for LRA. The patch
makes swap through registers instead of memory for the test cases when
LRA is used.
There are differences in reload and LRA constraint matching algorithm
which results in different alternative choices when the original pa
13 matches
Mail list logo
