On 11/27/13 10:30, Yvan Roux wrote:
Please include either the patch you are pinging or at the least a link to it
in the archives.
Ok, sorry for that, here is the patch and Changelog
Yvan
2013-11-17 Yvan Roux <yvan.r...@linaro.org>
* config/arm/arm.md (store_minmaxsi): Use only when
optimize_function_for_size_p.
Thanks.
This is fine for the trunk. And yes, having an insn's validity change
based on what block it's in is most definitely bad.
I was a bit concerned have the x86 backend, as I happened to know it
uses optimize_insn_for_* rather extensively. But thankfully it's only
used in splitters, expanders & peephole2 patterns, which should all be safe.
If you wouldn't mind, could you look at md.texi and see if there's a
reasonable place to put verbage about this issue in the internals
manual? It might save someone time in the future :-)
Thanks,
Jeff
