[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 pa...@matos-sorge.com * common.opt (fira-loop-pressure): Mark as optimization. OK to commit? -- PMatos

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 pa...@matos-sorge.com * common.opt (fira-loop-pressure): Mark as optimization. OK to commit? Patch is attached

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

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 fails here

[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 pa...@matos-sorge.com * c-family/c.opt: Add LTO FE

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

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 paulo.ma...@csr.com * regcprop.c (copy_value): remove check for redundant moves. * regcprop.c (copy_value): add check for redundant moves,

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 paulo.ma...@csr.com 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

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 pa...@matos-sorge.com wrote: Forgot to mention: this is to fix PR 54154. Updated changelog: 2012-08-02 Paulo Matos paulo.ma

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

Re: PATCH: Remove redundant instructions after regcprop

2012-08-02 Thread Paulo J. Matos
2012-08-02 Paulo Matos paulo.ma...@csr.com 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. ---

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 register_operand =c

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

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 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

[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 paulo.ma...@csr.com * gcc/rtlanal.c: Increase preference of a register for the first operand in a commutative

[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 paulo.ma...@csr.com 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

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 paulo.ma...@csr.com * 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

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 paulo.ma...@csr.com * 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