[patch][RFC] make lra.c:check_rtl set maybe_hot_insn_p

[Jeff Law]

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.roux@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