Hello,
The way to declare a inline virtual destructor within the class declaration
seems to unvirtualize it.
Trying the following code :
#include
struct A {
virtual ~A() = default;
};
struct B : A {
virtual ~B() {
std::cout << "B destructor" << std::endl;
}
};
int main() {
B* b = n
--- Comment #7 from spop at gcc dot gnu dot org 2010-02-06 07:14 ---
Subject: Bug 42637
Author: spop
Date: Sat Feb 6 07:13:39 2010
New Revision: 156538
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156538
Log:
Fix PR42637.
2010-02-05 Sebastian Pop
Konrad Trifun
--- Comment #11 from wilson at codesourcery dot com 2010-02-06 06:23
---
Subject: Re: Optimization flag -O1 -fschedule-insns2
cause red zone to be used when there is none
On Thu, 2010-02-04 at 10:48 +, rearnsha at gcc dot gnu dot org
wrote:
> I've been testing the attached patch
--- Comment #3 from howarth at nitro dot med dot uc dot edu 2010-02-06
05:34 ---
Index: gcc/config/darwin.h
===
--- gcc/config/darwin.h (revision 156530)
+++ gcc/config/darwin.h (working copy)
@@ -272,7 +272,7 @@
%{A}
--- Comment #2 from howarth at nitro dot med dot uc dot edu 2010-02-06
03:49 ---
Is this a typo in the committed patch?
-%(link_libgcc) %o %{fprofile-arcs|fprofile-generate*|coverage:-lgcov} \
to
+%{L*} %(link_libgcc) %o %{fprofile-arcs|fprofile-generate|coverage:-lgcov}
\
N
--- Comment #1 from howarth at nitro dot med dot uc dot edu 2010-02-06
03:46 ---
Could this be a side-effect of r155534...
2009-12-31 Iain Sandoe
PR target/41605
* config/darwin.h (LINK_COMMAND_SPEC): Resolve fopenmp specifically
for target.
(LINK_SPE
The gcc.dg/profile-generate-3.c test case no longer compiles on
*-apple-darwin*. On x86_64-apple-darwin10 the test case produces the error...
Executing on host:
/sw/src/fink.build/gcc45-4.4.999-20100205/darwin_objdir/gcc/xgcc
-B/sw/src/fink.build/gcc45-4.4.999-20100205/darwin_objdir/gcc/
/sw/src
--- Comment #4 from vmakarov at redhat dot com 2010-02-06 00:57 ---
I have a patch which solves the problem and analogous problem that Jeff
recently sent me.
I just need a time to do some benchmarking. If everything is all right, I'll
submit the patch probably on Monday.
--
http:/
--- Comment #3 from hubicka at ucw dot cz 2010-02-06 00:51 ---
Subject: Re: [4.5 regression] IRA apparently
systematically making reload too busy on 2 address instructions
with 3 operands
> The argument for using conflicts is that the vast majority of the time if the
>
--- Comment #8 from davek at gcc dot gnu dot org 2010-02-06 00:35 ---
just verified against r.156467
--
davek at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #7 from davek at gcc dot gnu dot org 2010-02-06 00:34 ---
closing old bug after reverifying with latest sources and patch for bug 42776
--
davek at gcc dot gnu dot org changed:
What|Removed |Added
---
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |paolo dot carlini at oracle
|dot org
--- Comment #1 from raj dot khem at gmail dot com 2010-02-06 00:06 ---
Created an attachment (id=19811)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19811&action=view)
testcase
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42981
gcc ICE's when compiling attached example
arm-none-eabi-gcc lib_pad.i -c -O -fPIC -mthumb
../ncurses/./base/lib_pad.c: In function pnoutrefresh:
../ncurses/./base/lib_pad.c:218: internal compiler error: in
default_secondary_reload, at targhooks.c:660
Please submit a full bug report,
with prepr
--- Comment #3 from longb at cray dot com 2010-02-05 23:49 ---
This bug was filed because it appeared that the test worked with the Cray, PGI,
and Intel compilers. However, based on the notes in Comments 1 and 2, more
tests were written which uncovered that the Cray compiler seems to do
--- Comment #5 from reza dot yazdani at amd dot com 2010-02-05 23:14
---
"The optimization performed is correct because:
In C++ (not in C) a const modifier in a global variable has internal
linkage (i.e. it is treated like a static variable) and therefore the
optimization performed is
--- Comment #4 from reza dot yazdani at amd dot com 2010-02-05 23:11
---
(In reply to comment #3)
> "The optimization performed is correct because:
>
> In C++ (not in C) a const modifier in a global variable has internal
> linkage (i.e. it is treated like a static variable) and therefo
--- Comment #2 from gary at intrepid dot com 2010-02-05 23:03 ---
(In reply to comment #1)
> Related to PR 33119. I don't many people who install with -j.
>
It resulted from "make install" being invoked from a Makefile, where
the overall make was run in parallel, and then a last step
--- Comment #8 from davek at gcc dot gnu dot org 2010-02-05 22:39 ---
(In reply to comment #7)
> Fixed.
>
Thanks!
--
davek at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #7 from jason at gcc dot gnu dot org 2010-02-05 22:38 ---
Fixed.
--
jason at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #1 from pinskia at gcc dot gnu dot org 2010-02-05 22:37 ---
Related to PR 33119. I don't many people who install with -j.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42980
--- Comment #6 from jason at gcc dot gnu dot org 2010-02-05 22:17 ---
Subject: Bug 42870
Author: jason
Date: Fri Feb 5 22:17:09 2010
New Revision: 156529
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156529
Log:
PR c++/42870
* config/i386/cygming.h (ASM_OUTPUT_
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Summary|"-fcompare-debug failure" |[4.5 Regression] "-fcompare-
|with "-O2 -finline-fu
--- Comment #35 from kkojima at gcc dot gnu dot org 2010-02-05 21:54
---
(In reply to comment #34)
> I was expecting that the landing pad was
> a good place for the constant pool on the contrary.
I thought so too. But on second thought, it'd be a bit surprising
for the non CP world a
While testing some mods to the current pre-4.5 tree,
I ran into sporadic "make install" failures, when
running the make install as a parallel make (ie,
"make -jN" where N > 1).
The host is an x86_64 with 4 CPU cores, being built in the default
multi-lib mode. The failures are sporadic, occurring
--- Comment #3 from reza dot yazdani at amd dot com 2010-02-05 21:10
---
"The optimization performed is correct because:
In C++ (not in C) a const modifier in a global variable has internal
linkage (i.e. it is treated like a static variable) and therefore the
optimization performed is
--- Comment #5 from paolo dot carlini at oracle dot com 2010-02-05 21:06
---
DR 342 has been resolved as NAD: time to return to this issue...
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
-
--- Comment #9 from jvdelisle at gcc dot gnu dot org 2010-02-05 20:06
---
Fixed.
--
jvdelisle at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSI
OpenMP #pragma omp taskwait after a label is not permitted - should fail at
compile time [Cray Bug 755983]
The test checks that the #pragma omp taskwait may not be used after a label.
Currently the GNU gcc compiler fails to produce any error message at
compile time.
The OpenMP API version 3 (May
--- Comment #3 from hjl dot tools at gmail dot com 2010-02-05 19:35 ---
It is due to X86_TUNE_INTER_UNIT_MOVES is off by default. I used pextrd
to avoid one more memory access since -mtune=core2 isn't faster than
default in most cases.
For this case, icc generates:
movl 4(
--- Comment #1 from simon at pushface dot org 2010-02-05 19:05 ---
Created an attachment (id=19810)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19810&action=view)
Report the switches passed to ranlib
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42978
When gnatmake is generating a static library, it uses ar (or the system
archiver) and ranlib (if found). In verbose mode it reports the commands
executed (in the case of ar, truncated if need be).
The reported command for ranlib doesn't include any options that may be
supplied: on this architectur
--- Comment #24 from paolo dot carlini at oracle dot com 2010-02-05 18:46
---
I see, great, just wanted to make sure the issue is recorded somewhere besides
this v3 PR.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42819
--- Comment #23 from jwakely dot gcc at gmail dot com 2010-02-05 18:40
---
I realised the other day I hadn't followed up on a fix for this.
Here's some of the email I got from Lawrence just before Christmas, I haven't
heard anything more recent:
> I noticed while implementing it that
--- Comment #13 from sje at cup dot hp dot com 2010-02-05 18:23 ---
Fixed with the definition of TARGET_DELEGITIMIZE_ADDRESS.
--
sje at cup dot hp dot com changed:
What|Removed |Added
--- Comment #12 from sje at gcc dot gnu dot org 2010-02-05 18:21 ---
Subject: Bug 42924
Author: sje
Date: Fri Feb 5 18:21:06 2010
New Revision: 156526
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156526
Log:
2010-02-05 Steve Ellcey
PR target/42924
* config
--- Comment #1 from zsojka at seznam dot cz 2010-02-05 18:14 ---
Created an attachment (id=19809)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19809&action=view)
most reduced testcase, still quite long
Command line:
g++ -O2 -finline-functions -fomit-frame-pointer -ftracer
-fsched
Command line:
g++ -O2 -finline-functions -fomit-frame-pointer -ftracer
-fsched2-use-superblocks -fPIC -fcompare-debug -c testcase.cpp -m32
(x86_64 needs -m32)
Tested revisions:
r156515 - crash
r156472 - crash
r156468 - crash
r156467 - OK
Testcase before reducing was crashing stopped crashing in r
--- Comment #2 from tromey at gcc dot gnu dot org 2010-02-05 18:04 ---
I tried this with some random svn trunk checkout, and it worked ok:
<1><25>: Abbrev Number: 2 (DW_TAG_namespace)
<26> DW_AT_name: bar
<2a> DW_AT_decl_file : 1
<2b> DW_AT_dec
--- Comment #4 from davek at gcc dot gnu dot org 2010-02-05 17:44 ---
Created an attachment (id=19808)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19808&action=view)
proposed fix
I'm going to bootstrap and test this patch, which adds a dummy static
dependency to pull in the nonc
--- Comment #3 from bkoz at gcc dot gnu dot org 2010-02-05 17:40 ---
Would love to see something standardized for all libs in gcc. My preference is
to have libstdc++ config docs linked to in gcc docs. (No duplication, just
alias.)
One thing that could be done would be to add a "C++ Spe
--- Comment #24 from paolo dot carlini at oracle dot com 2010-02-05 17:30
---
So you have to investigate why on your machine the GNU locale model is not
used, the configury falls back to the generic model. Certainly nothing changed
lately in this area, and, as you can see on testresults
--- Comment #23 from ghazi at gcc dot gnu dot org 2010-02-05 17:26 ---
(In reply to comment #22)
> Kaveh, you are comparing apples to oranges: in the first case the GNU locale
> model is used, a complete set of locale data is installed, thus the testcase
> is
> run and it fails. In the
--- Comment #2 from herumi at nifty dot com 2010-02-05 17:20 ---
>You should split your application into files that are compiled with either
-msse2 or -msse4. Using -msse4, you will get what you asked for.
I see, but according to Intel 64 and IA-32 Architectures Optimization
Reference M
--- Comment #22 from paolo dot carlini at oracle dot com 2010-02-05 17:04
---
Kaveh, you are comparing apples to oranges: in the first case the GNU locale
model is used, a complete set of locale data is installed, thus the testcase is
run and it fails. In the second case, evidently one
--- Comment #21 from ghazi at gcc dot gnu dot org 2010-02-05 16:55 ---
Sometime in Jan 2010 between revisions 155638 and 155826, this testcase stopped
failing on the trunk:
FAIL: http://gcc.gnu.org/ml/gcc-testresults/2010-01/msg00507.html
no FAIL: http://gcc.gnu.org/ml/gcc-testresults/
--- Comment #1 from raj dot khem at gmail dot com 2010-02-05 16:55 ---
after rebooting the machine. I dont see this happening. Closing invalid.
--
raj dot khem at gmail dot com changed:
What|Removed |Added
--
--- Comment #4 from pault at gcc dot gnu dot org 2010-02-05 16:28 ---
(In reply to comment #3)
> Subject: Re: ICE segfault when reading module file
>
> Tobias,
>
> I'll give it some thought at lunchtime.
I did give it some thought but got interrupted in writing my reply.
As long as
--- Comment #2 from law at redhat dot com 2010-02-05 16:20 ---
The argument for using conflicts is that the vast majority of the time if the
constraints force us to tie a specific source to the destination, then the
other source must not use the same hard reg as the destination. There m
//Nouse skiped
// test for holdkey
if(keybuf.keys!=keys.keys) {
20c:20 91 fa 00 ldsr18, 0x00FA
210:30 91 fb 00 ldsr19, 0x00FB
214:40 91 fc 00 ldsr20, 0x00FC
218:50 91 fd 00 ldsr21, 0x00FD
21c:80 91 04 01 ld
--- Comment #4 from mikpe at it dot uu dot se 2010-02-05 15:40 ---
Further experiments show that the failures are triggered by using the
binutils-2.20 version of ld when bootstrapping.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40860
--- Comment #5 from matz at gcc dot gnu dot org 2010-02-05 15:36 ---
The code for the if() looks sane on x86-64:
-
;; if (D.2729_8 != 0)
(insn 16 15 17 pr42972.c:10 (set (reg:QI 87)
(mem/s:QI (reg/f:DI 82 [ D.2727 ]) [0 S1 A32])) -1 (nil))
--- Comment #1 from rguenth at gcc dot gnu dot org 2010-02-05 15:34 ---
Mine anyway.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|una
$Summary.
Not possible with the patch from
http://gcc.gnu.org/ml/gcc-patches/2010-02/msg00235.html
because
g++.dg/lto/20081109 -m32 -fwhopr
FAILs again.
--
Summary: eh-cleanup should be able to clear
DECL_FUNCTION_PERSONALITY
Product: gcc
--- Comment #5 from rguenth at gcc dot gnu dot org 2010-02-05 15:32 ---
Fixed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #4 from rguenth at gcc dot gnu dot org 2010-02-05 15:31 ---
Subject: Bug 42762
Author: rguenth
Date: Fri Feb 5 15:31:15 2010
New Revision: 156520
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156520
Log:
2010-02-05 Richard Guenther
PR lto/42762
GNAT does not generate dynamic/runtime bounds checking for array slice
assignment.
Following code should raise exception (but it doesn't):
-- gnatmake -gnatf -gnato testcase.adb
procedure Testcase is
type Element_Access is access all Integer;
type Element_Array is
array (Positive range
--- Comment #5 from rguenth at gcc dot gnu dot org 2010-02-05 14:59 ---
Mine.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned
--- Comment #19 from steven at gcc dot gnu dot org 2010-02-05 14:58 ---
Interesting: for " -march=armv5te -mthumb" the code after IVOPTS is the perfect
code (from e.g. comment #17). The reason is that the address cost function for
Thumb (arm_thumb_address_cost) is of course not the same
--- Comment #19 from jamborm at gcc dot gnu dot org 2010-02-05 14:40
---
Created an attachment (id=19807)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19807&action=view)
Patch disallowing piecemeal and total scalarization in presence of volatile ops
I'm currently bootstrapping a
--- Comment #4 from rguenth at gcc dot gnu dot org 2010-02-05 14:23 ---
(In reply to comment #3)
> Subject: Re: Weird temporary array allocation
>
> On Fri, 5 Feb 2010, pault at gcc dot gnu dot org wrote:
>
> > --- Comment #2 from pault at gcc dot gnu dot org 2010-02-05 05:36
>
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.5.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42973
--- Comment #5 from davek at gcc dot gnu dot org 2010-02-05 14:06 ---
Patch posted:
http://gcc.gnu.org/ml/gcc-patches/2010-02/msg00156.html
Patched C++ testsuite results:
http://gcc.gnu.org/ml/gcc-testresults/2010-02/msg00414.html
One new failure observed:
http://gcc.gnu.org/ml/gcc-pat
--- Comment #18 from steven at gcc dot gnu dot org 2010-02-05 14:02 ---
I used "-O2 -std=c99 -mcpu=arm9 -funroll-loops" and I manually hacked the cost
in GDB to change from:
Address costs:
index costs 6
cst + index costs 2
...to this...:
Address costs:
index costs 1
cst + index c
--- Comment #17 from rakdver at kam dot mff dot cuni dot cz 2010-02-05
13:58 ---
Subject: Re: Inefficient loop unrolling
> But if I adjust the cost to make
> "index" cost only 1 or 2, I get this:
>
> :
> pretmp.11_26 = (int) s_11(D);
> ivtmp.25_28 = (long unsigned int) b_inout_5
--- Comment #16 from steven at gcc dot gnu dot org 2010-02-05 13:33 ---
I'm trying to coerce IVOPTSs into producing the following, optimal code in the
GIMPLE optimizers (without much luck, so far):
:
pretmp.11_26 = (int) s_11(D);
ivtmp.20_28 = (long unsigned int) b_inout_5(D);
D.1
--- Comment #4 from rguenth at gcc dot gnu dot org 2010-02-05 13:25 ---
(In reply to comment #1)
> In the .expand dump there is already something funny about the code generated
> for the bit field expressions.
>
> For example the code generated for this:
> D.1966_8 = D.1965_7 & 1;
>
--- Comment #5 from paolo dot carlini at oracle dot com 2010-02-05 13:20
---
Eric, Rainer, do you have any comment on this? For sure it's very hard to
provide an implementation working around the reported Solaris limitation
without breaking the binary compatibility. Does the limitation
--- Comment #3 from rguenth at gcc dot gnu dot org 2010-02-05 13:17 ---
On the (retired) mem-ref branch bitfield loads/stores were lowered very early
to read-extract-modify-write operations so the tree level would have optimized
this.
But of course people complained that architectures t
--- Comment #29 from paolo dot carlini at oracle dot com 2010-02-05 13:16
---
Given the resolution of DR 456 [CD1], this is invalid, that is, the snippet can
well compile (and does, with libstdc++-v3).
--
paolo dot carlini at oracle dot com changed:
What|Removed
--- Comment #2 from paolo dot carlini at oracle dot com 2010-02-05 13:12
---
Benjamin, any opinion about this issue?
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
--
--- Comment #19 from paolo dot carlini at oracle dot com 2010-02-05 13:11
---
So, this is fixed.
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
--- Comment #1 from hubicka at gcc dot gnu dot org 2010-02-05 13:11 ---
-funroll-instructions is shortcut for -O2 -funroll-loops -fno-schedule-insns2
(last one is there just to make asm prettier)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42973
Following testcase (derived from pattern appearing in internal loops of NAND
SPEC2k6 benchmark):
double a[1000];
double b[1000];
void t(void)
{
int i;
double carried = 0;
double inv=a[99];
for (i=0;i<1000;i++)
{
carried = inv-carried;
double tmp = carried *carried;
--- Comment #2 from jamborm at gcc dot gnu dot org 2010-02-05 13:08 ---
I wonder what to do about this. Perhaps I should change the test to
run only on i386, x86_64 and perhaps some other enumerated platforms?
That would be enough to alert us if SRA was failing in this regard and
would
--- Comment #8 from paolo dot carlini at oracle dot com 2010-02-05 13:07
---
It seems this can be safely closed as invalid, there is no reason why
std::__cos should be wrong.
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
--- Comment #7 from paolo dot carlini at oracle dot com 2010-02-05 12:57
---
At this point, I don't think we are going to do change the deprecated auto_ptr,
so closing as WONTFIX.
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
--- Comment #22 from paolo dot carlini at oracle dot com 2010-02-05 12:55
---
Jon, do we have some sort of reference / pointer for this issue? Are we sure
Lawrence is aware of it?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42819
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
Status|NEW |SUSPENDED
Summary|[C++0x] |[DR 887][C++0x]
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last re
--- Comment #11 from paolo dot carlini at oracle dot com 2010-02-05 12:52
---
Thus I'm resolving this as duplicate, if somebody disagrees, please re-open,
thanks.
*** This bug has been marked as a duplicate of 29131 ***
--
paolo dot carlini at oracle dot com changed:
Wh
--- Comment #9 from paolo dot carlini at oracle dot com 2010-02-05 12:52
---
*** Bug 31816 has been marked as a duplicate of this bug. ***
--
paolo dot carlini at oracle dot com changed:
What|Removed |Added
--- Comment #2 from steven at gcc dot gnu dot org 2010-02-05 12:45 ---
This is both a tree-optimization and an rtl-optimization problem, so setting
component to "middle-end" (there is no generic "optimization" component
anymore).
--
steven at gcc dot gnu dot org changed:
--- Comment #1 from steven at gcc dot gnu dot org 2010-02-05 12:42 ---
In the .expand dump there is already something funny about the code generated
for the bit field expressions.
For example the code generated for this:
D.1966_8 = D.1965_7 & 1;
if (D.1966_8 != 0)
TER will perform
--- Comment #16 from steven at gcc dot gnu dot org 2010-02-05 12:07 ---
Looks fixed on ARM. Pinski, what say you on ppc?
--
steven at gcc dot gnu dot org changed:
What|Removed |Added
-
--- Comment #3 from rguenth at gcc dot gnu dot org 2010-02-05 11:55 ---
I have a patch.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|
--- Comment #2 from rguenth at gcc dot gnu dot org 2010-02-05 11:36 ---
DECL_FUNCTION_PERSONALITY is set here because we see an EH tree at some point
that might need the C++ personality routine.
Note that only eh-cleanup removes this empty EH region, so at -O0 we'd see
DECL_FUNCTION_PER
Taken from http://hardwarebug.org/2010/01/30/bit-field-badness/
---
struct bf1_31 {
unsigned a:1;
unsigned b:31;
};
void func(struct bf1_31 *p, int n, int a)
{
int i = 0;
do {
if (p[i].a)
p[i].b += a;
} while (++i < n);
}
---
Inlining of guards is important for optimizing. Consider
int flag;
int foo (void)
{
if (!flag)
return 0;
...
}
we should be able to partially inline foo into its callers transforming
if (foo ())
{
...
}
else
{
...
}
to
if (!flag)
= 0;
e
IPA-SRA can do unused function parameter removal but misses unused return
value elimination. This can be useful for a function like
int
quantum_gate_counter(int inc)
{
static int counter = 0;
if(inc > 0)
counter += inc;
else if(inc < 0)
counter = 0;
return counter;
}
which is o
--- Comment #1 from rguenth at gcc dot gnu dot org 2010-02-05 10:52 ---
*** This bug has been marked as a duplicate of 42961 ***
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #1 from rguenth at gcc dot gnu dot org 2010-02-05 10:52 ---
*** Bug 42969 has been marked as a duplicate of this bug. ***
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42961
For following testcase:
int e,f;
float
main(float d,float c)
{
float a;
float e;
if (e)
a=d, e=c;
else
a=c, c=e;
if (f)
e=a;
return e;
}
(it looks artifical, but all the moves are there just to make "a" to be only
source or destination of reg-reg move that is not th
--- Comment #1 from steven at gcc dot gnu dot org 2010-02-05 10:44 ---
Be careful about naming RVCT.
--
steven at gcc dot gnu dot org changed:
What|Removed |Added
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Keywords||missed-optimization, ra
Target Milestone|---
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Severity|normal |enhancement
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42962
--- Comment #1 from rguenth at gcc dot gnu dot org 2010-02-05 10:15 ---
This optimization was triggered by the useless path which simplified some
jumps before CFG building which did the switch label merging.
--
rguenth at gcc dot gnu dot org changed:
What|Removed
--- Comment #1 from rguenth at gcc dot gnu dot org 2010-02-05 10:11 ---
You need to get ejc.jar by invoking contrib/download_ecj and specify
--with-ecj-jar=where/you/install/gcj.jar
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42967
--- Comment #4 from dodji at gcc dot gnu dot org 2010-02-05 09:36 ---
Indeed, the fix for PR 42915 in 4.5 makes this test case pass for me.
--
dodji at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #6 from dodji at gcc dot gnu dot org 2010-02-05 09:33 ---
Fixed in 4.5 (trunk)
--
dodji at gcc dot gnu dot org changed:
What|Removed |Added
Status
1 - 100 of 105 matches
Mail list logo
