This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

[PATCH] Fix get_maxval_strlen for COND_EXPR after changing them to ternary

From: Andrew Pinski <andrew dot pinski at caviumnetworks dot com>
To: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Wed, 16 May 2012 12:34:18 -0700
Subject: [PATCH] Fix get_maxval_strlen for COND_EXPR after changing them to ternary

When COND_EXPR was changed from GIMPLE_SINGLE_RHS to
GIMPLE_TERNARY_RHS, get_maxval_strlen was not updated for that
changed.  With a patch which has a late PHIOPT produce COND_EXPR, I
saw a couple of regressions (pr23484-chk.c and  strncpy-chk.c).

This patch fixes get_maxval_strlen for that change.

OK for the trunk and 4.7 branch?  Bootstrapped and tested on
x86_64-linux-gnu with no regressions.  And in 4.7.0 with the patch for
PHIOPT producing COND_EXPR on both x86_64-linux-gnu and mipsisa64-elf.

Thanks,
Andrew Pinski

ChangeLog:
* gimple-fold.c (get_maxval_strlen): Move COND_EXPR handling under
GIMPLE_ASSIGN.

Attachment: fixcondexprwithstrlen.diff.txt
Description: Text document

Follow-Ups:
- Re: [PATCH] Fix get_maxval_strlen for COND_EXPR after changing them to ternary
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]