Re: [PATCH] Mark fira-loop-pressure as optimization

2014-03-06 Thread Paulo J. Matos
On 06/03/14 23:34, Paulo J. Matos wrote: Hi, This patch marks fira-loop-pressure as an optimization so it shows in gcc --help=optimizers. 2014-03-06 Paulo Matos * common.opt (fira-loop-pressure): Mark as optimization. OK to commit? Patch is attached. -- PMatos Index: gcc

[PATCH] Mark fira-loop-pressure as optimization

2014-03-06 Thread Paulo J. Matos
Hi, This patch marks fira-loop-pressure as an optimization so it shows in gcc --help=optimizers. 2014-03-06 Paulo Matos * common.opt (fira-loop-pressure): Mark as optimization. OK to commit? -- PMatos

Re: [PATCH] [lto/55113] Fix use of -fshort-double with -flto for powerpc

2014-03-04 Thread Paulo J. Matos
On 04/03/14 11:16, Richard Biener wrote: It works fine on i386 for me but fails on x86_64. Please add a /* { dg-skip-if "PR-you-open" { { x86_64-*-* i?86-*-* } && lp64 } { "*" } { "" } } */ to the testcase to avoid the regression in the testsuite. Apologies, you're right. I meant x86_64 fai

Re: [PATCH] [lto/55113] Fix use of -fshort-double with -flto for powerpc

2014-03-04 Thread Paulo J. Matos
On 03/03/14 09:56, Richard Biener wrote: Index: gcc/c-family/c.opt === --- gcc/c-family/c.opt (revision 208249) +++ gcc/c-family/c.opt (working copy) @@ -1141,7 +1141,7 @@ C++ ObjC++ Optimization Var(flag_rtti) I Generate run tim

[PATCH] [lto/55113] Fix use of -fshort-double with -flto for powerpc

2014-03-01 Thread Paulo J. Matos
This patch fixes lto/55113 for powerpc. Combining -fshort-double with -flto is now working fine. I attach patch and testcase (unsure if testcase is in the right place). Tested with target powerpc-abispe. 2014-03-01 Paulo Matos * c-family/c.opt: Add LTO FE support for fshort-double

Re: Teaching emacs about GCC coding conventions (was Re: [PATCH] tree_code_name wrapper)

2013-10-16 Thread Paulo J. Matos
On 16/10/13 19:32, Mike Stump wrote: Committed revision 203715. Awesome! Can I backport your patch to 4_8? -- PMatos

Re: PATCH: Remove redundant instructions after regcprop

2012-08-02 Thread Paulo J. Matos
2012-08-02 Paulo Matos PR middle-end/54154 * regcprop.c (copy_value): remove check for redundant moves. * regcprop.c (copyprop_hardreg_forward_1): add check for redundant moves, remove instructions if redundant. --- //depot/bc/main/devHost/gcc46/gcc/gcc/regcprop.c 2011-09-06 1

Re: PATCH: Remove redundant instructions after regcprop

2012-08-02 Thread Paulo J. Matos
On 02/08/12 11:25, Steven Bosscher wrote: Hello, Thanks for working on this. How did you test this? This patch is for GCC46. The main problem is that I was not able to reproduce it (yet) for any upstream backends. I therefore patched GCC46 and tested our backend (where I can easily reprodu

Re: PATCH: Remove redundant instructions after regcprop

2012-08-02 Thread Paulo J. Matos
Thanks for the comments, I will be sending a new patch and fixed changelog. On 02/08/12 11:27, Richard Guenther wrote: On Thu, Aug 2, 2012 at 12:19 PM, Paulo J. Matos wrote: Forgot to mention: this is to fix PR 54154. Updated changelog: 2012-08-02 Paulo Matos PR middle-end/54154

Re: PATCH: Remove redundant instructions after regcprop

2012-08-02 Thread Paulo J. Matos
Forgot to mention: this is to fix PR 54154. Updated changelog: 2012-08-02 Paulo Matos PR middle-end/54154 * regcprop.c (copy_value): remove check for redundant moves. * regcprop.c (copy_value): add check for redundant moves, remove instructions if redundant.

PATCH: Remove redundant instructions after regcprop

2012-08-02 Thread Paulo J. Matos
Extended regcprop to check and remove for redundant move instructions resulting from the pass. Paulo. 2012-08-02 Paulo Matos * regcprop.c (copy_value): remove check for redundant moves. * regcprop.c (copy_value): add check for redundant moves, remove instructions if

Re: [PATCH] Prefer reg as first operand in commutative operator

2012-02-15 Thread Paulo J. Matos
On 14/02/12 14:10, Jakub Jelinek wrote: On Tue, Feb 14, 2012 at 02:05:30PM +, Paulo J. Matos wrote: I think the register allocator will generate good code using % if you make your predicate nonimmediate_operand in operand 1: (define_insn "iorqi3" [(set (match_operand:QI 0 "r

Re: [PATCH] Prefer reg as first operand in commutative operator

2012-02-14 Thread Paulo J. Matos
On 14/02/12 14:10, Jakub Jelinek wrote: It doesn't allow that, because the condition on the insn then fails, as neither operand 1 nor operand 2 is register_operand. Jakub Oh!!! I incorrectly read the instruction as I had initially tried it and I knew it was failing, however, I didn

Re: [PATCH] Prefer reg as first operand in commutative operator

2012-02-14 Thread Paulo J. Matos
On 14/02/12 13:46, Paolo Bonzini wrote: On 02/14/2012 10:52 AM, Jakub Jelinek wrote: > /* Complex expressions should be the first, so decrease priority > of objects. Prefer pointer objects over non pointer objects. */ > - if ((REG_P (op) && REG_POINTER (op)) > - || (MEM_P (op) && MEM_POINTER (op

Re: [PATCH] Prefer reg as first operand in commutative operator

2012-02-14 Thread Paulo J. Matos
On 14/02/12 09:52, Jakub Jelinek wrote: On Tue, Feb 14, 2012 at 07:27:21AM +, Paulo J. Matos wrote: case RTX_OBJ: /* Complex expressions should be the first, so decrease priority of objects. Prefer pointer objects over non pointer objects. */ - if ((REG_P (op

[PATCH] Prefer reg as first operand in commutative operator

2012-02-13 Thread Paulo J. Matos
Hi, This patch was submitted as part of PR 52235. It increases the preference of a register for first operand of a commutative operator. 2012-02-13 Paulo Matos * gcc/rtlanal.c: Increase preference of a register for the first operand in a commutative operator. --- gcc46/gcc/rt

[PATCH] PR rtl-optimization/49884: get_last_value checks register mode before returning value

2011-07-28 Thread Paulo J. Matos
-28 Paulo J. Matos PR rtl-optimization/49884 * combine.c (get_last_value): For registers whose value was set using a different mode than the one we are currently inquiring about we return 0. diff --git a/gcc/combine.c b/gcc/combine.c index 4dbf022..5885f2a 100644

Fix comment of get_last_value

2011-07-27 Thread Paulo J. Matos
There is a mistake in the comment for get_last_value in combine.c. This patch fixes this. PMatos 2011-07-27 Paulo J. Matos * Fix comment if get_last_value in combine.c. diff --git a/gcc/combine.c b/gcc/combine.c index 4dbf022..affb509 100644 --- a/gcc/combine.c +++ b/gcc/combine.c

Re: Fix typo in internal documents

2011-07-27 Thread Paulo J. Matos
Today is the 27th, not 26th so the Changelog should be: 2011-07-27 Paulo J. Matos * Fix internal documentation typo. TERGET should be TARGET. On 27/07/11 15:21, Paulo J. Matos wrote: There is a typo in the internal documentation. This patch fixes this. Please let me know if the patch

Fix typo in internal documents

2011-07-27 Thread Paulo J. Matos
There is a typo in the internal documentation. This patch fixes this. Please let me know if the patch is not in the required format. PMatos 2011-07-26 Paulo J. Matos * Fix internal documentation typo. TERGET should be TARGET. diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi index