Re: [Patch, ARM] PR71061, length pop* pattern in epilogue correctly

2016-06-09 Thread Kyrill Tkachov
Hi Jiong, On 02/06/16 15:44, Jiong Wang wrote: On 31/05/16 15:15, Kyrill Tkachov wrote: +/* Compute the atrribute "length" of insn. Currently, this function is used + for "*load_multiple_with_writeback", "*pop_multiple_with_return" and + "*pop_multiple_with_writeback_and_return". */ + s

Re: [Patch, ARM] PR71061, length pop* pattern in epilogue correctly

2016-06-02 Thread Jiong Wang
On 31/05/16 15:15, Kyrill Tkachov wrote: +/* Compute the atrribute "length" of insn. Currently, this function is used + for "*load_multiple_with_writeback", "*pop_multiple_with_return" and + "*pop_multiple_with_writeback_and_return". */ + s/atrribute/attribute/ Also, please add a descr

Re: [Patch, ARM] PR71061, length pop* pattern in epilogue correctly

2016-05-31 Thread Kyrill Tkachov
Hi Jiong, On 19/05/16 09:19, Jiong Wang wrote: On 13/05/16 14:54, Jiong Wang wrote: For thumb mode, this is causing wrong size calculation and may affect some rtl pass, for example bb-order where copy_bb_p needs accurate insn length info. This have eventually part of the reason for https://g

Re: [Patch, ARM] PR71061, length pop* pattern in epilogue correctly

2016-05-19 Thread Jiong Wang
On 13/05/16 14:54, Jiong Wang wrote: For thumb mode, this is causing wrong size calculation and may affect some rtl pass, for example bb-order where copy_bb_p needs accurate insn length info. This have eventually part of the reason for https://gcc.gnu.org/ml/gcc-patches/2016-05/msg00639.html

[Patch, ARM] PR71061, length pop* pattern in epilogue correctly

2016-05-13 Thread Jiong Wang
For thumb mode, this is causing wrong size calculation and may affect some rtl pass, for example bb-order where copy_bb_p needs accurate insn length info. This have eventually part of the reason for https://gcc.gnu.org/ml/gcc-patches/2016-05/msg00639.html where bb-order failed to do the bb copy.