diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2002-02-02 00:43:19 +0100 |
|---|---|---|
| committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2002-02-02 00:43:19 +0100 |
| commit | d3c52658629a1d170df67278db4c56e4a2742283 (patch) | |
| tree | b34fdba5de42b6c55a7fb0e29cafe34e997bdf78 | |
| parent | 02c5a3bd59e93557d9910c769d1c98100d84d6da (diff) | |
| download | gcc-d3c52658629a1d170df67278db4c56e4a2742283.zip gcc-d3c52658629a1d170df67278db4c56e4a2742283.tar.gz gcc-d3c52658629a1d170df67278db4c56e4a2742283.tar.bz2 | |
re PR c/5304 (gcc-20011231 generates incorrect divmod code for chars)
PR c/5304:
* expmed.c (expand_mult_highpart): Use immed_double_const for wide_op1
unconditionally.
* gcc.c-torture/execute/20020201-1.c: New test.
From-SVN: r49416
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/expmed.c | 15 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 4 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.c-torture/execute/20020201-1.c | 37 |
4 files changed, 53 insertions, 9 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 31aa4d7..1ba5874 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2002-02-02 Jakub Jelinek <jakub@redhat.com> + + PR c/5304: + * expmed.c (expand_mult_highpart): Use immed_double_const for wide_op1 + unconditionally. + 2002-02-01 Janis Johnson <janis187@us.ibm.com> * cfganal.c: Include tm_p.h. diff --git a/gcc/expmed.c b/gcc/expmed.c index 53ff05b..a26ad09 100644 --- a/gcc/expmed.c +++ b/gcc/expmed.c @@ -2760,15 +2760,12 @@ expand_mult_highpart (mode, op0, cnst1, target, unsignedp, max_cost) op1 = GEN_INT (trunc_int_for_mode (cnst1, mode)); - if (GET_MODE_BITSIZE (wider_mode) <= HOST_BITS_PER_INT) - wide_op1 = op1; - else - wide_op1 - = immed_double_const (cnst1, - (unsignedp - ? (HOST_WIDE_INT) 0 - : -(cnst1 >> (HOST_BITS_PER_WIDE_INT - 1))), - wider_mode); + wide_op1 + = immed_double_const (cnst1, + (unsignedp + ? (HOST_WIDE_INT) 0 + : -(cnst1 >> (HOST_BITS_PER_WIDE_INT - 1))), + wider_mode); /* expand_mult handles constant multiplication of word_mode or narrower. It does a poor job for large modes. */ diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 3be0340..1086bb1 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2002-02-02 Jakub Jelinek <jakub@redhat.com> + + * gcc.c-torture/execute/20020201-1.c: New test. + 2002-02-01 Janis Johnson <janis187@us.ibm.com> PR target/5469 diff --git a/gcc/testsuite/gcc.c-torture/execute/20020201-1.c b/gcc/testsuite/gcc.c-torture/execute/20020201-1.c new file mode 100644 index 0000000..b15f228 --- /dev/null +++ b/gcc/testsuite/gcc.c-torture/execute/20020201-1.c @@ -0,0 +1,37 @@ +/* Test whether division by constant works properly. */ + +extern void abort (void); +extern void exit (int); + +unsigned char cx = 7; +unsigned short sx = 14; +unsigned int ix = 21; +unsigned long lx = 28; +unsigned long long Lx = 35; + +int +main () +{ + unsigned char cy; + unsigned short sy; + unsigned int iy; + unsigned long ly; + unsigned long long Ly; + + cy = cx / 6; if (cy != 1) abort (); + cy = cx % 6; if (cy != 1) abort (); + + sy = sx / 6; if (sy != 2) abort (); + sy = sx % 6; if (sy != 2) abort (); + + iy = ix / 6; if (iy != 3) abort (); + iy = ix % 6; if (iy != 3) abort (); + + ly = lx / 6; if (ly != 4) abort (); + ly = lx % 6; if (ly != 4) abort (); + + Ly = Lx / 6; if (Ly != 5) abort (); + Ly = Lx % 6; if (Ly != 5) abort (); + + exit(0); +} |