Re: [PATCH] Fix thinko in handle_alias_pairs (PR c/61271)

2014-09-01 Thread Marek Polacek
On Tue, Aug 19, 2014 at 01:50:41PM +0200, Marek Polacek wrote: Sure, especially in the cgraph code... I'll wait until next week or so, thanks. I've backported to 4.8/4.9 now. Marek

[PATCH] Fix thinko in handle_alias_pairs (PR c/61271)

2014-08-19 Thread Marek Polacek
handle_alias_pairs contains the following thinko. I believe it should be not a virtual function. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2014-08-19 Marek Polacek pola...@redhat.com PR c/61271 * cgraphunit.c (handle_alias_pairs): Fix condition. diff --git

Re: [PATCH] Fix thinko in handle_alias_pairs (PR c/61271)

2014-08-19 Thread Richard Biener
On Tue, Aug 19, 2014 at 12:25 PM, Marek Polacek pola...@redhat.com wrote: handle_alias_pairs contains the following thinko. I believe it should be not a virtual function. Bootstrapped/regtested on x86_64-linux, ok for trunk? Ok. Is only trunk affected? Thanks, Richard. 2014-08-19 Marek

Re: [PATCH] Fix thinko in handle_alias_pairs (PR c/61271)

2014-08-19 Thread Marek Polacek
On Tue, Aug 19, 2014 at 01:22:20PM +0200, Richard Biener wrote: On Tue, Aug 19, 2014 at 12:25 PM, Marek Polacek pola...@redhat.com wrote: handle_alias_pairs contains the following thinko. I believe it should be not a virtual function. Bootstrapped/regtested on x86_64-linux, ok for trunk?

Re: [PATCH] Fix thinko in handle_alias_pairs (PR c/61271)

2014-08-19 Thread Richard Biener
On Tue, Aug 19, 2014 at 1:31 PM, Marek Polacek pola...@redhat.com wrote: On Tue, Aug 19, 2014 at 01:22:20PM +0200, Richard Biener wrote: On Tue, Aug 19, 2014 at 12:25 PM, Marek Polacek pola...@redhat.com wrote: handle_alias_pairs contains the following thinko. I believe it should be not a

Re: [PATCH] Fix thinko in handle_alias_pairs (PR c/61271)

2014-08-19 Thread Marek Polacek
On Tue, Aug 19, 2014 at 01:35:20PM +0200, Richard Biener wrote: On Tue, Aug 19, 2014 at 1:31 PM, Marek Polacek pola...@redhat.com wrote: On Tue, Aug 19, 2014 at 01:22:20PM +0200, Richard Biener wrote: On Tue, Aug 19, 2014 at 12:25 PM, Marek Polacek pola...@redhat.com wrote: