Re: [ping][PATCH] Power: Reorder a sign-extend RTL pattern for readability

2012-09-10 Thread Maciej W. Rozycki
On Sat, 8 Sep 2012, David Edelsohn wrote: > 2012-08-10 Maciej W. Rozycki > > gcc/ > * config/rs6000/rs6000.md: Move a splitter next to its insn. > > This patch is okay. Yes, the splitter should not have been separated > from the basic pattern. Thanks for helping to clean up the p

Re: [ping][PATCH] Power: Reorder a sign-extend RTL pattern for readability

2012-09-08 Thread David Edelsohn
While examining the Power MD file seeking the explanation for a problem I saw I have noticed a change in the past separated one of the instruction splitters from its corresponding instruction pattern. Several unrelated patterns were inserted between the two, presumably by accident where the `patc

[ping][PATCH] Power: Reorder a sign-extend RTL pattern for readability

2012-09-07 Thread Maciej W. Rozycki
Hi, This almost obviously correct proposal: http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00619.html received no feedback whatsoever and still awaits review. Would you please have a look at it? Thanks. Maciej

[PATCH] Power: Reorder a sign-extend RTL pattern for readability

2012-08-10 Thread Maciej W. Rozycki
Hi, While examining the Power MD file seeking the explanation for a problem I saw I have noticed a change in the past separated one of the instruction splitters from its corresponding instruction pattern. Several unrelated patterns were inserted between the two, presumably by accident where t