mirror/gcc
2
0
Fork 0
mirror of git://gcc.gnu.org/git/gcc.git synced 2025-03-16 17:21:07 +08:00
Code Issues Packages Projects Releases Wiki Activity

re PR rtl-optimization/11210 (optimizer drops conditional with typecast from signed to unsigned char)

Browse Source 
PR optimization/11210
	* fold-const (decode_field_reference): Strip only NOPs that
	don't affect the sign.

From-SVN: r68524
This commit is contained in:
Eric Botcazou 2003-06-26 12:37:35 +02:00 committed by Eric Botcazou
parent fc5eb4a111
commit 6dc208a1ed
4 changed files with 30 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
gcc/ChangeLog
View File

@ -1,3 +1,9 @@
2003-06-26 Eric Botcazou <ebotcazou@libertysurf.fr>
PR optimization/11210
* fold-const (decode_field_reference): Strip only NOPs that
don't affect the sign.
2003-06-26 Dhananjay Deshpande <dhananjayd@kpitcummins.com> 2003-06-26 Dhananjay Deshpande <dhananjayd@kpitcummins.com>
* gcc/config/sh/sh.md (push_fpscr): Enable for TARGET_SH2E. * gcc/config/sh/sh.md (push_fpscr): Enable for TARGET_SH2E.

3
gcc/fold-const.c
View File

@ -2742,7 +2742,8 @@ decode_field_reference (exp, pbitsize, pbitpos, pmode, punsignedp,
if (! INTEGRAL_TYPE_P (TREE_TYPE (exp))) if (! INTEGRAL_TYPE_P (TREE_TYPE (exp)))
return 0; return 0;
STRIP_NOPS (exp); /* Signedness matters here. */
STRIP_SIGN_NOPS (exp);
if (TREE_CODE (exp) == BIT_AND_EXPR) if (TREE_CODE (exp) == BIT_AND_EXPR)
{ {

4
gcc/testsuite/ChangeLog
View File

@ -1,3 +1,7 @@
2003-06-26 Eric Botcazou <ebotcazou@libertysurf.fr>
* gcc.dg/20030626-1.c: New test.
2003-06-26 Neil Booth <neil@daikokuya.co.uk> 2003-06-26 Neil Booth <neil@daikokuya.co.uk>
* const-str-2.m: Update. * const-str-2.m: Update.

18
gcc/testsuite/gcc.dg/20030626-1.c Normal file
View File

@ -0,0 +1,18 @@
/* PR optimization/11210 */
/* Originator: Guido Classen <guido@clagi.de> */
/* Reduced testcase by Falk Hueffner <falk@debian.org> */
/* { dg-do compile } */
/* { dg-options "-O" } */
/* Verify that the constant expressions folder doesn't
throw away the cast operation in the comparison. */
struct str {
int head;
char data[8];
};
int foo(struct str t)
{
return t.data[0] || (unsigned char) t.data[2] != 130; /* { dg-bogus "comparison is always 1" } */
}