I believe that the following is a bug. it is accepted by MSVC and comeau.
GCC seems to ADL-lookup names in namespaces that are only used in the template
arguments of the function arguments, for example:
A a;
adl_func(a); //otherns::adl_func is called
this can cause unintended conflicts with non-
--- Comment #1 from sstrasser at systemhaus-gruppe dot de 2010-05-29 07:45
---
a workaround (other than renaming the ADL function) would be appreciated
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44320
--- Comment #31 from iains at gcc dot gnu dot org 2010-05-29 07:47 ---
4.5-branch (as of r160013) has an error in config.gcc (which I just fixed on
trunk yesterday, with r159979) in which several t-make fragments are included
twice on x86_64-*-darwin*.
I wonder if this could be the ori
--- Comment #4 from iains at gcc dot gnu dot org 2010-05-29 07:49 ---
4.5-branch (as of r160013) has an error in config.gcc (which I just fixed on
trunk yesterday, with r159979) in which several t-make fragments are included
twice on x86_64-*-darwin*.
If anyone is interested to try it
--- Comment #3 from iains at gcc dot gnu dot org 2010-05-29 08:01 ---
(In reply to comment #2)
> Fixed after 159920 but before or including 159930. At closer inspection, it
> has to be r159929. :) On the other hand, from the patch message it seems it's
> just intended to be a stop-gap
--- Comment #4 from gcc at magfr dot user dot lysator dot liu dot se
2010-05-29 08:17 ---
I have run head-first into this problem as well.
I have found that _ZN1tcvT_IiEEv ( t::operator int() ) works but
_ZN1tcvKT_IiEEv ( t::operator int const() ) fails so it seems the problem
is that
--- Comment #2 from paolo dot carlini at oracle dot com 2010-05-29 09:30
---
Works fine in 4_5-branch and mainline, the behavior in 4_3-branch and
4_4-branch isn't a regression in any way, as fas as I can see.
--
paolo dot carlini at oracle dot com changed:
What|Remo
In the (to-be) attached testcase, GCC warns about an unused result when the
return value of the function is ignored, or cast to void, but it fails to warn
when the cast-to-void is hidden inside an inline function, like so:
$ cat -n wur.c
1
2 static inline void ignore_value (int i) { (vo
--- Comment #1 from davek at gcc dot gnu dot org 2010-05-29 11:33 ---
Created an attachment (id=20771)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20771&action=view)
testcase as per initial comment.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44321
--- Comment #8 from mikpe at it dot uu dot se 2010-05-29 11:35 ---
(In reply to comment #6)
> I note that copypage-xscale.c:xscale_mc_copy_user_highpage() calls a __naked
> function to do the bulk copy. Converting that to a plain inline function
> (changing 'pc' to 'lr' in the final ins
--- Comment #2 from rguenth at gcc dot gnu dot org 2010-05-29 11:40 ---
You can add
void bar4 (void)
{
int dummy;
dummy = foo ();
}
so I'm not sure the ignore_value () function call isn't a use. In fact
if you externalize that function it is at least a possible use.
Which also ra
--- Comment #9 from rguenth at gcc dot gnu dot org 2010-05-29 11:42 ---
(In reply to comment #8)
> (In reply to comment #6)
> > I note that copypage-xscale.c:xscale_mc_copy_user_highpage() calls a __naked
> > function to do the bulk copy. Converting that to a plain inline function
> > (
--- Comment #10 from rguenth at gcc dot gnu dot org 2010-05-29 11:47
---
Or rather, if you have
void __attribute__((naked)) foo (int i)
{
asm("use i");
}
without any inputs refering to i that is invalid. Like I see in the
attached preprocessed source:
static void __attribute__((na
--- Comment #11 from rguenth at gcc dot gnu dot org 2010-05-29 11:50
---
(it seems quite stupid to have naked functions with only an asm inside in the
first place - you can equally well use plain assembly)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44290
The following code is valid according to C99:
void * restrict const a[2];
void * restrict const (*p2)[2];
void foo(void) {
p2 = &a;
}
However GCC complains with:
a.c: In function 'foo':
a.c:14:7: warning: assignment from incompatible pointer type
If either "const" or "restrict" (or both) is
The naked attribute should cause two things noinline and noclone.
Sent from my iPhone
On May 29, 2010, at 4:50 AM, "rguenth at gcc dot gnu dot org" > wrote:
--- Comment #11 from rguenth at gcc dot gnu dot org 2010-05-29
11:50 ---
(it seems quite stupid to have naked functions wit
--- Comment #12 from pinskia at gmail dot com 2010-05-29 13:30 ---
Subject: Re: [4.5 Regression] arm linux kernel crahes when built with
-fipa-sra, __naked attribute is broken
The naked attribute should cause two things noinline and noclone.
Sent from my iPhone
On May 29, 2010, at 4:
--- Comment #4 from danglin at gcc dot gnu dot org 2010-05-29 14:16 ---
Subject: Bug 44261
Author: danglin
Date: Sat May 29 14:16:11 2010
New Revision: 160027
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160027
Log:
PR target/44261
config/pa/pa.md (negdf2_slow,
--- Comment #5 from bergner at gcc dot gnu dot org 2010-05-29 14:17 ---
Subject: Bug 44266
Author: bergner
Date: Sat May 29 14:17:26 2010
New Revision: 160028
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160028
Log:
Backport from mainline:
2010-05-28 Alan Mod
--- Comment #13 from mikpe at it dot uu dot se 2010-05-29 14:27 ---
(In reply to comment #10)
> Or rather, if you have
>
> void __attribute__((naked)) foo (int i)
> {
> asm("use i");
> }
>
> without any inputs refering to i that is invalid.
Not according to gcc/doc/extend.texi:
> @
--- Comment #5 from danglin at gcc dot gnu dot org 2010-05-29 14:33 ---
Fixed on trunk.
--
danglin at gcc dot gnu dot org changed:
What|Removed |Added
Status|
--- Comment #14 from mikpe at it dot uu dot se 2010-05-29 14:39 ---
(In reply to comment #11)
> (it seems quite stupid to have naked functions with only an asm inside in the
> first place - you can equally well use plain assembly)
Except that with plain asm() for an entire function defi
--- Comment #15 from rguenth at gcc dot gnu dot org 2010-05-29 14:45
---
(In reply to comment #13)
> (In reply to comment #10)
> > Or rather, if you have
> >
> > void __attribute__((naked)) foo (int i)
> > {
> > asm("use i");
> > }
> >
> > without any inputs refering to i that is in
--- Comment #16 from rguenth at gcc dot gnu dot org 2010-05-29 14:46
---
And CC a arm maintainer. There might be more target-specific attributes that
need adjustment.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #17 from mikpe at it dot uu dot se 2010-05-29 14:47 ---
Created an attachment (id=20772)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20772&action=view)
test case from copypage-xscale.c
This is distilled from the kernel's copypage-xscale.c file and illustrates the
iss
--- Comment #2 from hjl at gcc dot gnu dot org 2010-05-29 15:05 ---
Subject: Bug 44315
Author: hjl
Date: Sat May 29 15:05:30 2010
New Revision: 160029
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160029
Log:
Add a missing `\'.
2010-05-29 H.J. Lu
PR bootstrap/44315
--- Comment #5 from dave at hiauly1 dot hia dot nrc dot ca 2010-05-29
15:11 ---
Subject: Re: abi docs say that hppa-linux defaults to libgcc_s.so.2
> > Dave, when did the hppa-linux so version change from 2 to 4?
> > I'd like to document that, rather than just say it's always 1 or 4
>
On Linux/x86-64, IRA/reload moves asm statement by
adding "movss" at -O2:
[...@gnu-6 vzeroupper-1]$ cat foo.c
extern void bar2 (void);
float
foo (float y)
{
asm volatile ("nop");
bar2 ();
return y;
}
[...@gnu-6 vzeroupper-1]$ gcc -S -O foo.c
[...@gnu-6 vzeroupper-1]$ cat foo.s
.file
I don't think this is valid. You cannot depend on where the spill will
happen around a function call. It is spilling to save the volatile
register. With -O, we don't use volatile registers to keep variables
across functions. While at -O2 we do so it saves it right before the
function call.
--- Comment #1 from pinskia at gmail dot com 2010-05-29 15:39 ---
Subject: Re: New: IRA/reload moves asm statement
I don't think this is valid. You cannot depend on where the spill will
happen around a function call. It is spilling to save the volatile
register. With -O, we don't
--- Comment #2 from pinskia at gcc dot gnu dot org 2010-05-29 15:56 ---
There is no bug here since the asm does not say it touches any registers so
xmm0 is live across the asm. Reload/IRA can insert the spilling/saving before
or after the asm. Both places are valid.
--
pinskia at
--- Comment #3 from hjl dot tools at gmail dot com 2010-05-29 16:23 ---
A patch is posted at
http://gcc.gnu.org/ml/gcc-patches/2010-05/msg02325.html
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
---
On Linux/ia32, gcc.dg/tree-ssa/ipa-cp-1.c failed:
FAIL: gcc.dg/tree-ssa/ipa-cp-1.c scan-tree-dump-times optimized
"very_long_function.constprop.0 \(\)" 3
--
Summary: gcc.dg/tree-ssa/ipa-cp-1.c failed
Product: gcc
Version: 4.6.0
Status: UNCONFIRMED
--- Comment #1 from hjl dot tools at gmail dot com 2010-05-29 17:00 ---
It is caused by revision 160016:
http://gcc.gnu.org/ml/gcc-cvs/2010-05/msg01075.html
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
---
On Linux/ia64, revision 159926 gave:
FAIL: gcc.dg/vect/vect-109.c scan-tree-dump-times vect "vectorized 1 loops" 2
Revision 159909 is OK.
--
Summary: [4.6 regression] FAIL: gcc.dg/vect/vect-109.c
Product: gcc
Version: 4.6.0
Status: UNCONFIRMED
--- Comment #1 from hjl dot tools at gmail dot com 2010-05-29 17:05 ---
It may be caused by revision 159920:
http://gcc.gnu.org/ml/gcc-cvs/2010-05/msg00977.html
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
---
--- Comment #5 from spop at gcc dot gnu dot org 2010-05-29 17:14 ---
Subject: Bug 44306
Author: spop
Date: Sat May 29 17:14:31 2010
New Revision: 160031
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160031
Log:
Don't use unshare_expr when not necessary.
2010-05-29 Sebastian P
--- Comment #6 from spop at gcc dot gnu dot org 2010-05-29 17:15 ---
Fixed.
--
spop at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #3 from bonzini at gnu dot org 2010-05-29 17:32 ---
I don't think this bug is of any use. Unlike nonnull, unused return values do
not trigger undesirable optimizations and (as far as I can tell) cannot
possibly result in miscompilation.
This bug is indeed about a loophole,
--- Comment #4 from rguenth at gcc dot gnu dot org 2010-05-29 17:46 ---
(In reply to comment #3)
> I don't think this bug is of any use. Unlike nonnull, unused return values do
> not trigger undesirable optimizations and (as far as I can tell) cannot
> possibly result in miscompilation.
--- Comment #4 from mrs at gcc dot gnu dot org 2010-05-29 17:58 ---
I believe this has already been fixed by r160020.
--
mrs at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #18 from mikpe at it dot uu dot se 2010-05-29 18:00 ---
Created an attachment (id=20773)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20773&action=view)
linux kernel workaround for attribute naked breakage
This patch makes the Linux kernel add noinline and noclone att
--- Comment #2 from hubicka at gcc dot gnu dot org 2010-05-29 18:04 ---
Subject: Bug 44324
Author: hubicka
Date: Sat May 29 18:04:02 2010
New Revision: 160033
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160033
Log:
PR middle-end/44324
Modified:
trunk/gcc/ChangeL
--- Comment #3 from hjl dot tools at gmail dot com 2010-05-29 18:35 ---
Created an attachment (id=20774)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20774&action=view)
A reduced testcase
[...@gnu-32 delta]$ /export/gnu/import/rrs/159912/usr/bin/gcc -S -O3 pr44295.ii
pr44295.ii:
--- Comment #5 from hjl dot tools at gmail dot com 2010-05-29 18:40 ---
(In reply to comment #4)
> I believe this has already been fixed by r160020.
>
Revision 160020:
http://gcc.gnu.org/ml/gcc-cvs/2010-05/msg01079.html
doesn't fix this bug. In fact, it breaks bootstrap:
http://gcc.
implicit-zee.c has
zeinsn_list = VEC_alloc (rtx, heap, 8);
FOR_EACH_BB (curr_block)
{
FOR_BB_INSNS (curr_block, curr_insn)
{
if (!INSN_P (curr_insn))
continue;
type = for_each_rtx (&PATTERN (curr_insn),
is_set_with_extens
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
Target Milestone|--- |4.6.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44326
--- Comment #2 from danglin at gcc dot gnu dot org 2010-05-29 20:18 ---
This was introduced by the following change:
2010-05-12 Jan Hubicka
* cgraph.h (struct varpool_node): Add aux.
* varasm.c (find_decl_and_mark_needed): Force output of varpool nodes.
* var
--- Comment #6 from mrs at gcc dot gnu dot org 2010-05-29 20:26 ---
Subject: Bug 44315
Author: mrs
Date: Sat May 29 20:26:12 2010
New Revision: 160035
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160035
Log:
2010-05-29 H.J. Lu
PR bootstrap/44315
* Makefile.
--- Comment #7 from mrs at gcc dot gnu dot org 2010-05-29 20:30 ---
Ok, should be fixed now.
--
mrs at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #1 from tkoenig at gcc dot gnu dot org 2010-05-29 22:20 ---
Confirmed.
--
tkoenig at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCON
--- Comment #3 from danglin at gcc dot gnu dot org 2010-05-29 22:25 ---
Subject: Bug 44165
Author: danglin
Date: Sat May 29 22:24:59 2010
New Revision: 160038
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160038
Log:
PR target/44165
* config/pa/pa32-linux.h (CTO
--- Comment #4 from danglin at gcc dot gnu dot org 2010-05-29 22:29 ---
Fixed by patch.
--
danglin at gcc dot gnu dot org changed:
What|Removed |Added
Status|
This blocks moving finalizing the compilation unit last in
cp_write_global_declarations.
--
Summary: build_java_method_aliases should use cgraph_node-
>same_body aliases list
Product: gcc
Version: 4.6.0
Status: UNCONFIRMED
Notes:
1. the same issue occurs on x86 and x86_64 linux hosts, with a GCC
cross-compiler of the same version is built with the same options
2. the same issue occurs with GCC cross compiler 4.4.3 (same build options,
without the MPC host library)
I'm afraid I failed to understand something obvio
Currently gcc trunk is showing significant loss of vectorization at r160038 for
the gas_dyn.f90 benchmark...
bash-3.2$ gfortran -O3 -ffast-math -ftree-vectorizer-verbose=2 gas_dyn.f90 &>
gcc46.txt
bash-3.2$ grep "LOOP VECTORIZED" gcc46.txt | wc
29 1161177
compared to...
bash-3.2$ g
--- Comment #1 from howarth at nitro dot med dot uc dot edu 2010-05-30
00:34 ---
Sorry for the noise. Wrong version of gcc was being executed.
--
howarth at nitro dot med dot uc dot edu changed:
What|Removed |Added
--- Comment #10 from iains at gcc dot gnu dot org 2010-05-30 00:56 ---
Created an attachment (id=20775)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20775&action=view)
with -lto
the code @ comment # compiled with -flto -dA -fverbose-asm
--
http://gcc.gnu.org/bugzilla/show_b
--- Comment #11 from iains at gcc dot gnu dot org 2010-05-30 00:57 ---
Created an attachment (id=20776)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20776&action=view)
without lto
code from comment #5 compiled with -dA -fverbose-asm
--
http://gcc.gnu.org/bugzilla/show_bug.cg
--- Comment #12 from iains at gcc dot gnu dot org 2010-05-30 01:00 ---
OK, so I've patch obj-act.c to remove the two cases where ASM_ was called.
The test code compiles and runs without -flto and produces the errors as per
comment #9 when -flto is given.
The required content is pre
--- Comment #2 from jvdelisle at gcc dot gnu dot org 2010-05-30 05:38
---
I would like to work this one.
--
jvdelisle at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #19 from raj dot khem at gmail dot com 2010-05-30 05:58 ---
(In reply to comment #18)
> Created an attachment (id=20773)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20773&action=view) [edit]
> linux kernel workaround for attribute naked breakage
>
> This patch makes
--- Comment #5 from bonzini at gnu dot org 2010-05-30 06:42 ---
Richi, I think we're saying the same thing from two different directions.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44321
--- Comment #20 from raj dot khem at gmail dot com 2010-05-30 06:54 ---
Created an attachment (id=20777)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20777&action=view)
includes fix for all arms
I see the problem after reading Richard's comment #10, here is some more
additions to
64 matches
Mail list logo
