On January 16, 2020 6:08:48 AM GMT+01:00, apin...@marvell.com wrote: >From: Andrew Pinski <apin...@marvell.com> > >Commit g:f96bf49a0 added the target field to expand_operand. >But it leaves it uninitialized when doing a full initialization >inside create_expand_operand. This fixes the problem and improves >the code generation inside create_expand_operand too. > >OK? Bootstrapped and tested on x86_64-linux-gnu with no regressions.
OK. Richard. >ChangeLog: >* optabs.h (create_expand_operand): Initialize target field also. > >Change-Id: Ib653fbfbb2b0709970db87fb94de14b59758bc6c >--- > gcc/optabs.h | 1 + > 1 file changed, 1 insertion(+) > >diff --git a/gcc/optabs.h b/gcc/optabs.h >index 07bdc56586e..5bd19503a0a 100644 >--- a/gcc/optabs.h >+++ b/gcc/optabs.h >@@ -78,6 +78,7 @@ create_expand_operand (class expand_operand *op, > { > op->type = type; > op->unsigned_p = unsigned_p; >+ op->target = 0; > op->unused = 0; > op->mode = mode; > op->value = value;