This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PGO bootstrap on x390x (PR bootstrap/78985).
- From: Martin Liška <mliska at suse dot cz>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Cc: Richard Biener <richard dot guenther at gmail dot com>
- Date: Mon, 30 Jan 2017 14:32:48 +0100
- Subject: Re: [PATCH] Fix PGO bootstrap on x390x (PR bootstrap/78985).
- Authentication-results: sourceware.org; auth=none
- References: <08fd2591-2e9b-7b62-aa85-f2ef57efa947@suse.cz>
On 01/30/2017 12:27 PM, Martin Liška wrote:
> Hi.
>
> Following patch simply fixes issues reported by -Wmaybe-unitialized. That enables PGO bootstrap
> on a s390x machine.
>
> Ready to be installed?
> Martin
>
There's second version that adds one more hunk for s390 target.
Martin
>From 598d0a59b91070211b09056195bc0f971bc57ae1 Mon Sep 17 00:00:00 2001
From: marxin <mliska@suse.cz>
Date: Mon, 30 Jan 2017 11:09:29 +0100
Subject: [PATCH] Fix PGO bootstrap on x390x (PR bootstrap/78985).
gcc/ChangeLog:
2017-01-30 Martin Liska <mliska@suse.cz>
PR bootstrap/78985
* config/s390/s390.c (s390_gimplify_va_arg): Initialize local
variable to NULL.
(print_operand_address): Initialize a struct to zero.
---
gcc/config/s390/s390.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/gcc/config/s390/s390.c b/gcc/config/s390/s390.c
index fe65846a4f2..c02da273d8c 100644
--- a/gcc/config/s390/s390.c
+++ b/gcc/config/s390/s390.c
@@ -7347,6 +7347,7 @@ void
print_operand_address (FILE *file, rtx addr)
{
struct s390_address ad;
+ memset (&ad, 0, sizeof (s390_address));
if (s390_loadrelative_operand_p (addr, NULL, NULL))
{
@@ -12195,7 +12196,7 @@ s390_gimplify_va_arg (tree valist, tree type, gimple_seq *pre_p,
tree f_gpr, f_fpr, f_ovf, f_sav;
tree gpr, fpr, ovf, sav, reg, t, u;
int indirect_p, size, n_reg, sav_ofs, sav_scale, max_reg;
- tree lab_false, lab_over;
+ tree lab_false, lab_over = NULL_TREE;
tree addr = create_tmp_var (ptr_type_node, "addr");
bool left_align_p; /* How a value < UNITS_PER_LONG is aligned within
a stack slot. */
--
2.11.0