public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Jeff Law <law@redhat.com>
To: "Martin Liška" <mliska@suse.cz>, "GCC Patches" <gcc-patches@gcc.gnu.org>
Cc: Martin Sebor <msebor@gmail.com>, Jakub Jelinek <jakub@redhat.com>
Subject: Re: [PATCH] Fix warnings for profiledbootstrap (PR bootstrap/77788)
Date: Wed, 05 Oct 2016 16:41:00 -0000	[thread overview]
Message-ID: <4cfcf9fb-c17d-ed85-23b9-84d71b0997e2@redhat.com> (raw)
In-Reply-To: <a42502f2-1f69-8fcf-5c5b-57d84b747cde@suse.cz>

On 10/04/2016 03:16 AM, Martin Liška wrote:
> Hello.
>
> In the attached patch I resolve couple of warnings seen during make profiledbootstrap.
> I can survive bootstrap on both ppc64le-redhat-linux and x86_64-linux-gnu targets.
>
> Ready to be installed?
> Martin
>
>
> 0001-Fix-warnings-for-make-profiledbootstrap-PR-bootstrap.patch
>
>
> From 510ac204f2e448d244c8eec5ab8aeab85f6041a9 Mon Sep 17 00:00:00 2001
> From: marxin <mliska@suse.cz>
> Date: Mon, 3 Oct 2016 23:17:05 +0200
> Subject: [PATCH] Fix warnings for make profiledbootstrap (PR bootstrap/77788)
>
> gcc/ChangeLog:
>
> 2016-10-04  Martin Liska  <mliska@suse.cz>
>
> 	PR bootstrap/77788
> 	* expmed.h (mul_highpart_cost_ptr): Add an gcc_assert.
> 	* gimple-ssa-strength-reduction.c (slsr_process_cast):
> 	Initialize a pointer to NULL.
> 	(slsr_process_copy): Likewise.
> 	* input.c (location_get_source_line): Likewise.
> 	* tree-ssa-ccp.c (optimize_atomic_bit_test_and): Likewise.
Presumably what's going on here is that the profiling data is giving the 
jump threader additional information and the threader is refusing to 
thread some jumps because they're not profitable.  AT least that's how 
it looks to me.

Note the input.c change may not be necessary once the input.c cleanups 
are done, but I don't think it'd be actively harmful.

OK.

jeff

  reply	other threads:[~2016-10-05 16:41 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-04  9:17 Martin Liška
2016-10-05 16:41 ` Jeff Law [this message]
2016-10-07  8:25 ` [PATCH] Fix profiled bootstrap (part 2) Martin Liška
2016-10-06  6:34 [PATCH] Fix warnings for profiledbootstrap (PR bootstrap/77788) Bernd Edlinger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4cfcf9fb-c17d-ed85-23b9-84d71b0997e2@redhat.com \
    --to=law@redhat.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=jakub@redhat.com \
    --cc=mliska@suse.cz \
    --cc=msebor@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).