On 18 June 2014 at 11:06, Ramana Radhakrishnan <ramana....@googlemail.com> wrote: > On Tue, Jun 17, 2014 at 4:03 PM, Charles Baylis > <charles.bay...@linaro.org> wrote: >> Your mention of larger vector modes prompted me to check that the >> patch has the desired result with them. In fact, the costs are >> estimated incorrectly which means the post_modify pattern is not used. >> The attached patch fixes that. (used in combination with my original >> patch) >> >> >> 2014-06-15 Charles Baylis <charles.ba...@linaro.org> >> >> * config/arm/arm.c (arm_new_rtx_costs): Reduce cost for mem with >> embedded side effects. > > I'm not too thrilled with putting in more special cases that are not > table driven in there. Can you file a PR with some testcases that show > this so that we don't forget and CC me on it please ?
I created https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61551 at the time. I've come back to look at this again and would like to fix it in this release cycle. I still don't really understand what you mean by table-driven in this context. Do you still hold this view, and if so, could you describe what you'd like to see instead of this patch?
