Re: [PATCH][MIPS] Enable load-load/store-store bonding

2015-05-13 Thread sameera
Hi Mike, Thanks for your comments. Please find my comments inlined. - Thanks and regards, Sameera D. On Monday 11 May 2015 10:09 PM, Mike Stump wrote: On May 11, 2015, at 4:05 AM, sameera sameera.deshpa...@imgtec.com wrote: +(define_insn *join2_loadhi + [(set (match_operand:SI 0

Re: [PATCH][MIPS] Enable load-load/store-store bonding

2015-05-11 Thread sameera
On Tuesday 21 April 2015 12:39 AM, Matthew Fortune wrote: Sameera Deshpande sameera.deshpa...@imgtec.com writes: Gentle reminder! Thanks Sameera. Just a couple of comments inline below and a question for Catherine at the end. - Thanks and regards, Sameera D. On Monday 30 March 2015

RE: [PATCH][MIPS] Enable load-load/store-store bonding

2015-05-11 Thread Matthew Fortune
Hi Sameera, Sameera Deshpande sameera.deshpa...@imgtec.com writes: Changelog: gcc/ * config/mips/mips.md (JOIN_MODE): New mode iterator. (join2_load_StoreJOIN_MODE:mode): New pattern. (join2_loadhi): Likewise. (define_peehole2): Add peephole2 patterns to

Re: [PATCH][MIPS] Enable load-load/store-store bonding

2015-05-11 Thread sameera
On Monday 11 May 2015 05:43 PM, Matthew Fortune wrote: Hi Sameera, Sameera Deshpande sameera.deshpa...@imgtec.com writes: Changelog: gcc/ * config/mips/mips.md (JOIN_MODE): New mode iterator. (join2_load_StoreJOIN_MODE:mode): New pattern. (join2_loadhi): Likewise.

Re: [PATCH][MIPS] Enable load-load/store-store bonding

2015-05-11 Thread Mike Stump
On May 11, 2015, at 4:05 AM, sameera sameera.deshpa...@imgtec.com wrote: +(define_insn *join2_loadhi + [(set (match_operand:SI 0 register_operand =r) + (any_extend:SI (match_operand:HI 1 non_volatile_mem_operand m))) + (set (match_operand:SI 2 register_operand =r) + (any_extend:SI

RE: [PATCH][MIPS] Enable load-load/store-store bonding

2015-04-20 Thread Moore, Catherine
-Original Message- From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] Sent: Monday, April 20, 2015 3:10 PM To: Sameera Deshpande; Moore, Catherine Cc: Richard Sandiford; gcc-patches@gcc.gnu.org; echri...@gmail.com Subject: RE: [PATCH][MIPS] Enable load-load/store-store

Re: [PATCH][MIPS] Enable load-load/store-store bonding

2015-04-20 Thread Mike Stump
With FUSION you might get farther. See the arm port as I recall. The quick overview, FUSION allows instructions that are not contiguous to be paired up and fused together. it was built for load/load store/store combining. On Apr 19, 2015, at 10:09 PM, sameera sameera.deshpa...@imgtec.com

RE: [PATCH][MIPS] Enable load-load/store-store bonding

2015-04-20 Thread Matthew Fortune
Sameera Deshpande sameera.deshpa...@imgtec.com writes: Gentle reminder! Thanks Sameera. Just a couple of comments inline below and a question for Catherine at the end. - Thanks and regards, Sameera D. On Monday 30 March 2015 04:58 PM, sameera wrote: Hi! Sorry for delay in sending

Re: [PATCH][MIPS] Enable load-load/store-store bonding

2015-04-19 Thread sameera
Gentle reminder! - Thanks and regards, Sameera D. On Monday 30 March 2015 04:58 PM, sameera wrote: Hi! Sorry for delay in sending this patch for review. Please find attached updated patch. In P5600, 2 consecutive loads/stores of same type which access contiguous memory locations are

Re: [PATCH][MIPS] Enable load-load/store-store bonding

2015-03-30 Thread sameera
Hi! Sorry for delay in sending this patch for review. Please find attached updated patch. In P5600, 2 consecutive loads/stores of same type which access contiguous memory locations are bonded together by instruction issue unit to dispatch single load/store instruction which accesses both

RE: [PATCH][MIPS] Enable load-load/store-store bonding

2014-06-24 Thread Sameera Deshpande
Hi Richard, Thanks for the review. Please find attached updated patch after your review comments. Changelog: gcc/ * config/mips/mips.md (JOIN_MODE): New mode iterator. (join2_load_StoreJOIN_MODE:mode): New pattern. (join2_loadhi): Likewise. (define_peehole2): Add

RE: [PATCH][MIPS] Enable load-load/store-store bonding

2014-06-23 Thread Sameera Deshpande
Hi Richard, Thanks for your comments. I am working on the review comments, and will share the reworked patch soon. However, here is clarification on some of the issues raised. + if (TARGET_FIX_24K TUNE_P5600) +error (unsupported combination: %s, -mtune=p5600 -mfix-24k); + /*

Re: [PATCH][MIPS] Enable load-load/store-store bonding

2014-06-23 Thread Richard Sandiford
Sameera Deshpande sameera.deshpa...@imgtec.com writes: + if (TARGET_FIX_24K TUNE_P5600) +error (unsupported combination: %s, -mtune=p5600 -mfix-24k); + /* Save the base compression state and process flags as though we were generating uncompressed code. */

Re: [PATCH][MIPS] Enable load-load/store-store bonding

2014-06-21 Thread Richard Sandiford
Hi Sameera, Thanks for the patch. Sameera Deshpande sameera.deshpa...@imgtec.com writes: diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c index b5b5ba7..9804ef2 100644 --- a/gcc/config/mips/mips.c +++ b/gcc/config/mips/mips.c @@ -18813,6 +18813,9 @@ mips_option_override (void)

[PATCH][MIPS] Enable load-load/store-store bonding

2014-06-19 Thread Sameera Deshpande
Hi Richard, Please find attached the patch implementing load-load/store-store bonding supported by P5600. In P5600, 2 consecutive loads/stores of same type which access contiguous memory locations are bonded together by instruction issue unit to dispatch single load/store instruction which