On 11 September 2012 09:46, Michael Hope <michael.h...@linaro.org> wrote: > On 10 September 2012 18:47, Zhenqiang Chen <zhenqiang.c...@linaro.org> wrote: >> Summary: >> * Test shrink-wrap code >> >> Details: >> 1. Add simple_return support in function thumb2_expand_return for >> shrink-wrap. Here is the make check status >> * One new fail is due to code size increase. We'd disable it when >> optimizing function for size on THUMB2. >> * Other new fails is due to dwarf info. Root cause is ICE at function >> maybe_record_trace_start >> >> gcc_checking_assert (cfi_row_equal_p (cur_row, ti->beg_row)); >> >> Here is the failed code segment: >> >> tst ... L1 >> push {r4} >> ... >> ldr r4, ... >> L1: >> bx lr // common simple return from two branches. > > Hmm, any chance of a conditional return instead in the future?
ARM mode had supported conditional return. We can update current patterns or add new instruction patterns to support it for THUMB mode. I will work on it later. Thanks! -Zhenqiang _______________________________________________ linaro-toolchain mailing list linaro-toolchain@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-toolchain