Hi Richard,
Many thanks for the suggestion of an alternative implementation. I tried
implementing the suggestion, and I had a couple of observations:
1. As well as applying the bias in compute_fn_summary, it seemed to also
be necessary to apply it in ip_update_overall_fn_summary to avoid an ICE
r
This patch is an RFC to invite comments as to whether the approach
to solving the problem is a suitable one for upstream GCC, or whether
there are alternative approaches that would be recommended.
Motivation
--
We have observed that in some cases the estimation of callee growth for
inlini
Hi Jeff,
On 09/09/2019 19:55, Jeff Law wrote:
> I'm not sure if we really want to expose this as a first class option.
> Perhaps a PARAM would be better. Jan would have the final call though
> since he's done the most work on the IPA bits.
Many thanks for the suggestion - I could turn it into a
Since the combine pass canonicalises shift-add insns using plus and
ashift (as opposed to plus and mult which it previously used to do), it
no longer creates *add_n or *sub_n insns, as the patterns match plus and
mult only. The outcome of this is that some opportunities to generate
add{1,2,3} and s
Zissulescu wrote:
> Hi Graham,
>
> May I ask if you tested this with gcc's dejagnu?
>
> Thanks,
> Claudiu
>
> On Fri, May 12, 2017 at 9:14 PM, Graham Markall
> wrote:
>> Since the combine pass canonicalises shift-add insns using plus and
>> ashift (as oppos
assignment on file. As I don't have write access to GCC, if
this patch is approved, could someone apply it please (once the
corresponding binutils patch is approved and applied)?
Many thanks,
Graham Markall.
2016-12-12 Graham Markall
* config/arc/arc.h (LINK_SPEC): Use arclinu
Hi Claudiu,
On 12/12/16 19:18, Claudiu Zissulescu wrote:
> It looks sane to me, please apply (ask Andrew to do it for you)
>
> //Claudiu
Many thanks for the quick review. I'll ask Andrew to do it once the
binutils patch is approved and applied.
Best regards,
Graham.
signature.asc
Descriptio
