--- Comment #9 from jacob dot benoit dot 1 at gmail dot com 2009-12-11
17:10 ---
I also confirm that I still have the ICE with the .ii file attached to this bug
report.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42225
--- Comment #10 from hjl dot tools at gmail dot com 2009-12-11 17:24
---
Confirmed.
--
hjl dot tools at gmail dot com changed:
What|Removed |Added
Status|RES
--- Comment #11 from dodji at gcc dot gnu dot org 2009-12-11 22:12 ---
Hah. It looks like there are several slightly different problems at play here.
Apparently I just fixed one of them, which I was the one I narrowed in the
reduced test case above.
Then I forgot to try my fix on the ini
--- Comment #12 from dodji at gcc dot gnu dot org 2009-12-11 22:47 ---
Created an attachment (id=19278)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19278&action=view)
Draft patch
Here is the patch I am testing.
It seems to be working fine so far.
Note that g++ says the code is n
--- Comment #13 from jacob dot benoit dot 1 at gmail dot com 2009-12-11
22:51 ---
(In reply to comment #12)
> Note that g++ says the code is not valid though. Did you expect it to compile
> without error ?
The .ii file I have attached? Yes, I was expecting it to be correct, it
compiles
--- Comment #14 from hjl at gcc dot gnu dot org 2009-12-11 23:38 ---
Subject: Bug 42225
Author: hjl
Date: Fri Dec 11 23:36:24 2009
New Revision: 155184
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=155184
Log:
Backport testcases from trunk.
2009-12-11 H.J. Lu
Backp
--- Comment #15 from dodji at gcc dot gnu dot org 2009-12-14 08:24 ---
(In reply to comment #13)
> The .ii file I have attached? Yes, I was expecting it to be correct, it
> compiles without error or warnings with GCC 4.4, even with all warnings
> enabled, AFAIK.
Fair enough. It was me d
--- Comment #16 from jacob dot benoit dot 1 at gmail dot com 2009-12-14
12:43 ---
(In reply to comment #15)
> Fair enough. It was me doing weird things.
>
> I submitted a patch to http://gcc.gnu.org/ml/gcc-patches/2009-12/msg00669.html
> Let's see how it turns out.
>
> Thanks for your
--- Comment #17 from rguenth at gcc dot gnu dot org 2009-12-14 13:03
---
(In reply to comment #16)
> I mention this because I heared that GCC has a list of project it builds
> before
> releasing a version, as real-world test cases. Maybe Eigen's test suite can be
> added to that list?
--- Comment #18 from dodji at gcc dot gnu dot org 2009-12-19 22:40 ---
Subject: Bug 42225
Author: dodji
Date: Sat Dec 19 22:40:37 2009
New Revision: 155363
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=155363
Log:
Fix PR c++/42225, take 2
gcc/cp/ChangeLog:
PR c++/42225
--- Comment #19 from dodji at gcc dot gnu dot org 2009-12-19 22:43 ---
This should now be fixed in trunk for 4.5
--
dodji at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #20 from jacob dot benoit dot 1 at gmail dot com 2009-12-21
18:59 ---
Thank you, I can confirm that it's fixed here.
However I now hit another ICE, see bug #42450, but it seems unrelated.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42225
--- Comment #21 from hjl at gcc dot gnu dot org 2009-12-22 18:18 ---
Subject: Bug 42225
Author: hjl
Date: Tue Dec 22 18:17:53 2009
New Revision: 155403
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=155403
Log:
Backport testcases from mainline.
2009-12-22 H.J. Lu
Ba
--- Comment #2 from rguenth at gcc dot gnu dot org 2009-11-30 10:32 ---
With checking enabled I get
> g++-4.5 -S -o /dev/null product_small.ii -B /abuild/rguenther/trunk-g/gcc
In file included from /home/bjacob/eigen/test/product_small.cpp:26:0:
/home/bjacob/eigen/test/product.h: In fun
--
dodji at gcc dot gnu dot org changed:
What|Removed |Added
CC|dodji at gcc dot gnu dot org|
AssignedTo|unassigned at gcc dot gnu |dodji at gcc dot gnu d
--- Comment #3 from dodji at gcc dot gnu dot org 2009-12-01 00:26 ---
A reduced test case seems to be:
~=~
template
struct A
{
typedef T I;
};
template
struct B
{
typedef T TT;
typedef typename TT::I TT_I;
typedef A TA;
};
template
void
foo()
{
typedef T TT
--- Comment #4 from hjl dot tools at gmail dot com 2009-12-01 00:27 ---
It is caused by revision 145440:
http://gcc.gnu.org/ml/gcc-cvs/2009-04/msg00060.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42225
--- Comment #5 from dodji at gcc dot gnu dot org 2009-12-03 14:07 ---
A patch was submitted to
http://gcc.gnu.org/ml/gcc-patches/2009-12/msg00194.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42225
--- Comment #6 from dodji at gcc dot gnu dot org 2009-12-11 14:36 ---
Subject: Bug 42225
Author: dodji
Date: Fri Dec 11 14:36:05 2009
New Revision: 155160
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=155160
Log:
Fix PR c++/42225
gcc/cp/ChangeLog:
PR c++/42225
--- Comment #7 from dodji at gcc dot gnu dot org 2009-12-11 14:37 ---
Fixed in trunk (4.5)
--
dodji at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #8 from jacob dot benoit dot 1 at gmail dot com 2009-12-11
17:07 ---
(In reply to comment #7)
> Fixed in trunk (4.5)
>
Thanks for fixing this bug!
Maybe I'm just ignorant, but I can't get the fix to work for me. I have SVN
r155167. I am still getting a ICE, albeit at a sl
21 matches
Mail list logo