Source:
int f1(short a, int b)
{
return a * b;
}
int f2(unsigned short a, int b)
{
return a * b;
}
> gcc -O3 -fomit-frame-pointer -S paramext.c
_f1:
LFB0:
movl%esi, %eax
movswl %di, %edi <-
imull %edi, %eax
ret
...
_f2:
LFB1:
movl%esi, %
--- Comment #4 from pinskia at gcc dot gnu dot org 2010-06-14 07:35 ---
> if ((unsigned int)(xCount + 32) < 0x8000)
Yes even though the comparision is done in an unsigned type, the addition is
done in a signed type which means the overflow of that addition is undefined.
So cha
--- Comment #1 from jakub at gcc dot gnu dot org 2010-06-14 08:01 ---
tempreg = convert_to_mode (data->nominal_mode, tempreg, unsignedp);
if (GET_CODE (tempreg) == SUBREG
&& GET_MODE (tempreg) == data->nominal_mode
&& REG_P (SUBREG_REG (tempreg))
--- Comment #16 from jakub at gcc dot gnu dot org 2010-06-14 08:43 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
On x86_64-apple-darwin10 the polyhedron test capacita.f90 is miscompiled with
'-O3 -finline-limit=600 -flto' at revision 160679 (revision 160678 is fine):
[macbook] lin/test% gfcp -O3 -finline-limit=600 -flto capacita.f90
[macbook] lin/test% a.out
Give: N_x, N_y, GridSize_X, GridSize_Y, DoCircle
--- Comment #10 from jakub at gcc dot gnu dot org 2010-06-14 08:54 ---
In sel_print* case I wonder why it is defined as macros at all, it surely
bloats sel-sched* a lot for something that isn't enabled by default (verbose
dumps).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44426
the avx built-in _builtin_ia32_vextractf128_si256(X,N) acts as if the
immediate N was always set to 1.
to isolate the problem I compiled 2 programs:
---
#include
int main()
{
__m256i c = _mm256_set_epi32(1, 2, 3, 4, 5, 6, 7, 8);
__m128i d = (__m128i)__bui
g++/trunk/x86/rev.160690 -O0 compile test w/o errors, but with -O[ 123]
generates undefined symbol
# g++ -O0 -c 2010_06_13.cpp && nm -uAC 2010_06_13.o | grep OnProv
# g++ -O -c 2010_06_13.cpp && nm -uAC 2010_06_13.o | grep OnProv
2010_06_13.o: U FOO::A::OnProv()
# cat 2010_06_13.cpp
names
--- Comment #1 from dimhen at gmail dot com 2010-06-14 10:19 ---
simplified testcase FAIl with -O[23]
namespace FOO {
template
class A
{
public:
void Enum();
virtual void OnProv() = 0;
virtual ~A() { }
};
typedef A B;
template
void A::Enum ()
{
OnProv ();
}
} // names
--- Comment #11 from jakub at gcc dot gnu dot org 2010-06-14 10:20 ---
Patch for sel_print posted:
http://gcc.gnu.org/ml/gcc-patches/2010-06/msg01400.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44426
--- Comment #50 from davek at gcc dot gnu dot org 2010-06-14 10:38 ---
Subject: Bug 42776
Author: davek
Date: Mon Jun 14 10:38:18 2010
New Revision: 160722
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160722
Log:
ChangeLog:
Backport from mainline:
2010-04-27 Dave Korn
--- Comment #18 from dominiq at lps dot ens dot fr 2010-06-14 11:15 ---
With the patch in comment #17, x86_64-apple-darwin10 bootstrapped without
problem. Thanks.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44509
--- Comment #2 from rguenth at gcc dot gnu dot org 2010-06-14 11:23 ---
Confirmed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCON
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Component|c |target
Ever
--- Comment #2 from rguenth at gcc dot gnu dot org 2010-06-14 11:30 ---
Confirmed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCON
--- Comment #1 from ubizjak at gmail dot com 2010-06-14 11:35 ---
It looks to me that it is cut'n'pasto. All AVX vec_extract_lo_* should have
their immediate operand changed to 0x0.
--
ubizjak at gmail dot com changed:
What|Removed |Added
-
--- Comment #9 from irar at gcc dot gnu dot org 2010-06-14 12:22 ---
Subject: Bug 44507
Author: irar
Date: Mon Jun 14 12:22:13 2010
New Revision: 160727
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160727
Log:
PR tree-optimization/44507
* tree-vect-loop.c (ge
--- Comment #19 from jakub at gcc dot gnu dot org 2010-06-14 12:31 ---
Subject: Bug 44509
Author: jakub
Date: Mon Jun 14 12:30:29 2010
New Revision: 160729
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160729
Log:
PR bootstrap/44509
* c-config-lang.in (gtfiles):
--- Comment #20 from dominiq at lps dot ens dot fr 2010-06-14 12:32 ---
Could the patch in comment #17 explain the following failures?
FAIL: gcc.dg/torture/builtin-math-5.c -O0 scan-tree-dump-times original
"cexpf" 2
FAIL: gcc.dg/torture/builtin-math-5.c -O0 scan-tree-dump-times ori
--- Comment #21 from jakub at gcc dot gnu dot org 2010-06-14 12:32 ---
Fixed, sorry for the breakage.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #15 from jamborm at gcc dot gnu dot org 2010-06-14 12:39
---
(In reply to comment #14)
> SSE performance is fine again, thanks a lot!
>
> One more question, if that's OK...
> Depending on ARRSZ the testcase uses wildly varying amounts of CPU time; it's
> about half a second
--- Comment #6 from jamborm at gcc dot gnu dot org 2010-06-14 12:44 ---
Patch submitted to the mailing list:
http://gcc.gnu.org/ml/gcc-patches/2010-06/msg01146.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43905
--- Comment #16 from martin at mpa-garching dot mpg dot de 2010-06-14
12:46 ---
(In reply to comment #15)
I have found the problem in the meantime ... it's my mistake, sorry about the
noise :(
The problem is that I did not explicitly zero the arrays in main(), so they
apparently conta
--- Comment #13 from pluto at agmk dot net 2010-06-14 12:48 ---
do you plan to backport this fix to 4.5 branch?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44258
--- Comment #17 from jamborm at gcc dot gnu dot org 2010-06-14 12:50
---
OK, I did not put much effort into my thinking about it :-)
Yes, the testcase is fine as it is.
I'm not testing the patch on the 4.5 branch and will commit it today
if everything goes fine.
--
http://gcc.gnu.o
--- Comment #14 from jamborm at gcc dot gnu dot org 2010-06-14 12:51
---
(In reply to comment #13)
> do you plan to backport this fix to 4.5 branch?
>
Of course, I'm running the bootstrap and testsuite right now. I will
commit it today if everything goes fine.
--
http://gcc.gnu.
--- Comment #53 from howarth at nitro dot med dot uc dot edu 2010-06-14
13:14 ---
Now that r160722, the COFF lto patches, are committed to gcc 4.5 branch, we are
clear to backport r159173 as well for the mach-o patches. I've done this
locally and posted the testsuite results at
http://g
--- Comment #10 from irar at gcc dot gnu dot org 2010-06-14 13:43 ---
Subject: Bug 44507
Author: irar
Date: Mon Jun 14 13:43:24 2010
New Revision: 160742
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160742
Log:
PR tree-optimization/44507
* tree-vect-loop.c (ge
--- Comment #31 from dodji at gcc dot gnu dot org 2010-06-14 14:13 ---
Subject: Re: __extension__ keyword doesn't suppress warning on LL or ULL
constants
"manu at gcc dot gnu dot org" writes:
>> Next stop is to disable this feature by default, and enable it with a
>> -ftrack-macro-ex
--- Comment #32 from dodji at gcc dot gnu dot org 2010-06-14 14:20 ---
Created an attachment (id=20907)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20907&action=view)
Refreshed version.
- Disables the feature by default
- Adds a -ftrack-macro-expansion flag to enable the feature
the following should give an error message like:
fortcom: Error: test.f90, line 5: Since the OpenMP DEFAULT(NONE) clause
applies, the PRIVATE, SHARED, REDUCTION, FIRSTPRIVATE, or LASTPRIVATE attribute
must be explicitly specified for every variable. [A]
J=A(I)
^
compilation aborted for tes
--- Comment #2 from jakub at gcc dot gnu dot org 2010-06-14 15:19 ---
Subject: Bug 44508
Author: jakub
Date: Mon Jun 14 15:19:04 2010
New Revision: 160749
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160749
Log:
PR tree-optimization/44508
* tree-ssa-propagate.h
--- Comment #3 from jakub at gcc dot gnu dot org 2010-06-14 15:24 ---
Subject: Bug 44508
Author: jakub
Date: Mon Jun 14 15:23:31 2010
New Revision: 160750
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160750
Log:
PR tree-optimization/44508
* tree-ssa-propagate.h
--- Comment #4 from jakub at gcc dot gnu dot org 2010-06-14 15:28 ---
Subject: Bug 44508
Author: jakub
Date: Mon Jun 14 15:28:00 2010
New Revision: 160752
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160752
Log:
PR tree-optimization/44508
* tree-ssa-propagate.c
The assembler instructions gcc generates for a 'goto' statement only respect
the low 16 bits of the destination address by default (or in when using PIC in
general). This causes the program to jump to the wrong location and soon
thereafter segfault. The -mno-explicit-relocs seems to work around thi
--- Comment #1 from wesley at terpstra dot ca 2010-06-14 15:53 ---
Created an attachment (id=20908)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20908&action=view)
Pre-processed example file with bad branches.
Compile as described in the initial bug report.
--
http://gcc.gnu
--- Comment #5 from jakub at gcc dot gnu dot org 2010-06-14 15:53 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #12 from jakub at gcc dot gnu dot org 2010-06-14 15:54 ---
Subject: Bug 44426
Author: jakub
Date: Mon Jun 14 15:53:38 2010
New Revision: 160754
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160754
Log:
PR bootstrap/44426
* tree.h (build_call_expr): D
--- Comment #13 from jakub at gcc dot gnu dot org 2010-06-14 16:01 ---
Subject: Bug 44426
Author: jakub
Date: Mon Jun 14 16:00:39 2010
New Revision: 160755
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160755
Log:
PR bootstrap/44426
* tree.h (build_call_expr): D
Since the PR43949 fix was backported to 4_5-branch I'm seeing
FAIL: gcc.dg/vect/slp-perm-5.c scan-tree-dump-times vect "vectorized 1 loops" 1
FAIL: gcc.dg/vect/slp-perm-6.c scan-tree-dump-times vect "vectorized 1 loops" 1
in testsuite results on powerpc64-linux. They also fail on trunk since PR4
--- Comment #2 from pinskia at gcc dot gnu dot org 2010-06-14 16:34 ---
What version of as are you using? Because the assembler does some
optimizations/changes the load address and it looks like rather an assembler
issue rather than a compiler one.
--
pinskia at gcc dot gnu dot org
--- Comment #3 from wesley at terpstra dot ca 2010-06-14 16:40 ---
(sid)terps...@gabrielli:~/mlton/mlton-20100608$ as -v
GNU assembler version 2.20.1 (mips-linux-gnu) using BFD version (GNU Binutils
for Debian) 2.20.1-system.20100303
... is the %got($L894) supposed to cover the case whe
--- Comment #2 from hjl dot tools at gmail dot com 2010-06-14 17:05 ---
A patch is posted at
http://gcc.gnu.org/ml/gcc-patches/2010-06/msg01443.html
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
---
--- Comment #4 from wesley at terpstra dot ca 2010-06-14 17:22 ---
Created an attachment (id=20909)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20909&action=view)
Libraries needed to demonstrate the problem in a linked program
To be able to run the complete program:
gcc -std=gn
--- Comment #5 from wesley at terpstra dot ca 2010-06-14 17:25 ---
In case it is a binutils problem, I've attached a '.a' file sufficient to fully
link the program. To get to the problem jump is pretty easy:
gdb ./mlyacc.bad
break Chunk6
run
s
OUTPUT:
2926 nextFun = 2736;
(gd
--- Comment #2 from nightstrike at gmail dot com 2010-06-14 17:35 ---
I can confirm this as a valid bug, but I don't have the bugzilla permissions to
do so. Can someone update this to New?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44455
--- Comment #3 from hjl at gcc dot gnu dot org 2010-06-14 17:44 ---
Subject: Bug 44534
Author: hjl
Date: Mon Jun 14 17:44:29 2010
New Revision: 160756
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160756
Log:
Replace 0x1 with 0x0 in AVX vec_extract_lo_* patterns.
gcc/
2010-06
Command line:
$ gcc -O[12] -ftracer -freorder-blocks testcase.c
It doesn't crash when -fno-ipa-pure-const is supplied.
Compiler output:
$ gcc -O1 -ftracer -freorder-blocks testcase.c
testcase.c: In function 'foo':
testcase.c:21:6: error: type mismatch between an SSA_NAME and its symbol
testcase.c
--- Comment #4 from hjl at gcc dot gnu dot org 2010-06-14 17:46 ---
Subject: Bug 44534
Author: hjl
Date: Mon Jun 14 17:46:12 2010
New Revision: 160757
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160757
Log:
Replace 0x1 with 0x0 in AVX vec_extract_lo_* patterns.
gcc/
2010-06
--- Comment #1 from zsojka at seznam dot cz 2010-06-14 17:48 ---
Created an attachment (id=20910)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20910&action=view)
reduced testcase (from gcc/lto-wrapper.c)
Command line:
$ gcc -O1 -ftracer -freorder-blocks pr44539.c
Bootstrap with
--- Comment #3 from froydnj at gcc dot gnu dot org 2010-06-14 17:51 ---
Switched to NEW for NightStrike.
--
froydnj at gcc dot gnu dot org changed:
What|Removed |Added
Command line:
$ g++ g++.dg/other/canon-33194.C -fkeep-inline-functions
Compiler output:
$ g++ g++.dg/other/canon-33194.C -fkeep-inline-functions
/mnt/svn/gcc-trunk/gcc/testsuite/g++.dg/other/canon-33194.C: In member function
'void dwflpp::translate_location()':
/mnt/svn/gcc-trunk/gcc/testsuite/g
--- Comment #5 from hjl at gcc dot gnu dot org 2010-06-14 18:07 ---
Subject: Bug 44534
Author: hjl
Date: Mon Jun 14 18:07:13 2010
New Revision: 160758
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160758
Log:
Replace 0x1 with 0x0 in AVX vec_extract_lo_* patterns.
gcc/
2010-06
--- Comment #6 from hjl dot tools at gmail dot com 2010-06-14 18:09 ---
Fixed.
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
Status|NEW
--- Comment #3 from changpeng dot fang at amd dot com 2010-06-14 18:28
---
Actually, the prefetching is for the following loop:
for (i = 0; i < p[2]; i++)
q[i] = 0;
I do not understand why unrolling of this loop affects other part of
the program that has longjmp.
--
http
--- Comment #3 from ebotcazou at gcc dot gnu dot org 2010-06-14 18:55
---
Presumably.
--
ebotcazou at gcc dot gnu dot org changed:
What|Removed |Added
URL
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.6.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44540
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.6.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44539
--- Comment #1 from rguenth at gcc dot gnu dot org 2010-06-14 19:02 ---
Confirmed. Initialization loops should get asm(""); markers.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #11 from rguenth at gcc dot gnu dot org 2010-06-14 19:03
---
Fixed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #8 from aoliva at gcc dot gnu dot org 2010-06-14 19:18 ---
Subject: Bug 43656
Author: aoliva
Date: Mon Jun 14 19:18:04 2010
New Revision: 160761
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160761
Log:
PR debug/43656
* haifa-sched.c (setup_insn_reg_pressure_info,
u
--- Comment #4 from aoliva at gcc dot gnu dot org 2010-06-14 19:18 ---
Subject: Bug 43650
Author: aoliva
Date: Mon Jun 14 19:18:18 2010
New Revision: 160762
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160762
Log:
PR debug/43650
PR debug/44181
PR debug/44247
* tree-ssa-loop-ma
--- Comment #6 from aoliva at gcc dot gnu dot org 2010-06-14 19:18 ---
Subject: Bug 44181
Author: aoliva
Date: Mon Jun 14 19:18:18 2010
New Revision: 160762
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160762
Log:
PR debug/43650
PR debug/44181
PR debug/44247
* tree-ssa-loop-ma
--- Comment #3 from aoliva at gcc dot gnu dot org 2010-06-14 19:18 ---
Subject: Bug 44247
Author: aoliva
Date: Mon Jun 14 19:18:18 2010
New Revision: 160762
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160762
Log:
PR debug/43650
PR debug/44181
PR debug/44247
* tree-ssa-loop-ma
--- Comment #9 from aoliva at gcc dot gnu dot org 2010-06-14 19:19 ---
Fixed
--
aoliva at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED
--- Comment #7 from aoliva at gcc dot gnu dot org 2010-06-14 19:22 ---
Fixed
--
aoliva at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED
--- Comment #1 from hjl dot tools at gmail dot com 2010-06-14 21:33 ---
It is caused by revision 159596:
http://gcc.gnu.org/ml/gcc-cvs/2010-05/msg00649.html
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
---
--- Comment #2 from hjl dot tools at gmail dot com 2010-06-14 21:39 ---
It is caused by revision 160124:
http://gcc.gnu.org/ml/gcc-cvs/2010-06/msg00036.html
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
---
--- Comment #4 from changpeng dot fang at amd dot com 2010-06-14 22:22
---
There is nothing wrong in the prefetch itself. The problem is
__builtin_prefetch call used for prefetch instruction. Whenever,
there is a non-local lable in the current function, the __builtin_prefetch
inserted
--- Comment #3 from hjl dot tools at gmail dot com 2010-06-15 00:57 ---
We should consider:
1. The x86-64 psABI doesn't say how char/short should be extended
as function parameters.
2. Gcc may not touch upper bits, PR 42324.
3. Gcc never depends on the upper bits which are properly exte
--- Comment #2 from aoliva at gcc dot gnu dot org 2010-06-15 02:48 ---
Mine
--
aoliva at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at
--- Comment #3 from aoliva at gcc dot gnu dot org 2010-06-15 02:51 ---
Created an attachment (id=20911)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20911&action=view)
Patch that fixes the problem
Testing this now.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44248
--- Comment #8 from danglin at gcc dot gnu dot org 2010-06-15 03:35 ---
Things appear to go wrong in the loop2_invariant pass. Prior to this pass,
the 'f' flag is not set in the rtx for ivtmp.314. The first occurence is here:
(insn 1154 1153 1155 103 ../../gcc/gcc/gimple-pretty-print.
73 matches
Mail list logo
