http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
John David Anglin changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #24 from John David Anglin 2012-08-18
23:16:57 UTC ---
Author: danglin
Date: Sat Aug 18 23:16:53 2012
New Revision: 190505
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=190505
Log:
PR middle-end/53823
* expmed.c (expa
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #23 from Richard Henderson 2012-08-13
15:51:37 UTC ---
On 08/12/2012 07:30 AM, danglin at gcc dot gnu.org wrote:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
>
> --- Comment #22 from John David Anglin
> 2012-08-12 14:30:12 U
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #22 from John David Anglin 2012-08-12
14:30:12 UTC ---
Created attachment 27994
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27994
Patch
/* synth_mult does an `unsigned int' multiply. As long as the mode is
les
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #21 from John David Anglin 2012-08-01
18:44:04 UTC ---
The issue is with the handling of negative constants.
In this bit of code,
max_cost = (set_src_cost (gen_rtx_MULT (mode, fake_reg, op1), speed)
-
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #20 from dave.anglin at bell dot net 2012-08-01 14:27:30 UTC ---
On 1-Aug-12, at 10:20 AM, dave.anglin at bell dot net wrote:
> The difference in extracts and deposits may not be the problem. The -
> O2 code appears to have the same e
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #19 from dave.anglin at bell dot net 2012-08-01 14:20:49 UTC ---
On 31-Jul-12, at 10:25 PM, rth at gcc dot gnu.org wrote:
> The cross-compile *ought* not to affect costs, which means that
> we ought to be making the same algorithm choi
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #18 from Richard Henderson 2012-08-01
02:25:03 UTC ---
But this is what you get when cross-compiling from i686.
3267 rtx temp = expand_mult_const (mode, op0, -coeff, NULL_RTX,
(gdb) p algorithm
$1 = {cost = {cost = 24, latency =
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #17 from John David Anglin 2012-08-01
00:40:24 UTC ---
(gdb) p *(const struct algorithm *)$r17
$21 = {cost = {cost = 24, latency = 24}, ops = 7, op = {alg_m, alg_sub_t2_m,
alg_add_factor, alg_sub_t2_m, alg_sub_t2_m, alg_add_t2_m,
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
John David Anglin changed:
What|Removed |Added
Status|ASSIGNED|NEW
--- Comment #16 from John David A
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #15 from dave.anglin at bell dot net 2012-07-31 22:55:54 UTC ---
Created attachment 27916
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27916
zz.s.txt
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #14 from dave.anglin at bell dot net 2012-07-31 22:54:59 UTC ---
On 31-Jul-12, at 5:29 PM, rth at gcc dot gnu.org wrote:
> How does this -O1 output compare with native?
There seems to an off by one error in various shifts, etc. For
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #13 from Richard Henderson 2012-07-31
21:29:52 UTC ---
Created attachment 27914
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27914
Cross-compile output
Hmm. I can't seem to reproduce this via a cross-compiler.
I tried CC='gc
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #12 from dave.anglin at bell dot net 2012-07-31 20:10:55 UTC ---
> Mine.
Expected result for testcase is 112975202 (0x6bbdd62). Miscompiled
result is
116077092194 (0x1b06bbdd62).
--
John David Anglindave.ang...@bell.net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
Richard Henderson changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|unassigne
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #10 from John David Anglin 2012-07-31
19:41:20 UTC ---
Created attachment 27913
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27913
Testcase
Testcase produces wrong output when compiled at -O0 and -O1.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #9 from John David Anglin 2012-07-31
18:51:26 UTC ---
Regarding the remaining acats fails, the following line in a-calend.adb
appears to be miscompiled:
Date_Dur := Date_Dur - Time_Dur (Four_Year_Segs) *
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #8 from John David Anglin 2012-07-30
15:46:13 UTC ---
Author: danglin
Date: Mon Jul 30 15:46:08 2012
New Revision: 189980
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=189980
Log:
PR middle-end/53823
* expmed.c (expan
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #7 from Uros Bizjak 2012-07-30 14:25:32
UTC ---
(In reply to comment #6)
> And this is why "uninitialized" warnings shouldn't be silenced like this...
>
> * expmed.c (expand_mult): Initialize coeff and is_neg.
>
> http://gcc.gnu
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
Steven Bosscher changed:
What|Removed |Added
CC||uros at gcc dot gnu.org
--- Comment #6
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
Richard Henderson changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #4 from John David Anglin 2012-07-29
16:31:22 UTC ---
Created attachment 27888
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27888
Patch
Fixes 930921-1.c and four original acats fails, but not
additional acats fails introduced
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #3 from John David Anglin 2012-07-27
18:19:26 UTC ---
*** Bug 53974 has been marked as a duplicate of this bug. ***
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
--- Comment #2 from John David Anglin 2012-07-12
01:43:11 UTC ---
This hunk of RTL was generated in .expand for function f:
;; D.1356_3 = D.1355_2 * 2863311531;
(insn 9 8 10 (set (reg:SI 104)
(subreg:SI (reg:DI 96 [ D.1355 ]) 4))
/test/g
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
John David Anglin changed:
What|Removed |Added
CC||rth at gcc dot gnu.org
--- Comment #1
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53823
Richard Guenther changed:
What|Removed |Added
Target Milestone|--- |4.8.0
26 matches
Mail list logo
