diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index 118d25d8f56d..7b5da5349cd4 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,8 @@ +2008-01-28 Jason Merrill + + PR c++/35007 + * class.c (build_base_path): Fix !want_pointer case. + 2008-01-27 Jason Merrill PR c++/27177 diff --git a/gcc/cp/class.c b/gcc/cp/class.c index 0ce4ba499905..9f7d986676cc 100644 --- a/gcc/cp/class.c +++ b/gcc/cp/class.c @@ -296,7 +296,12 @@ build_base_path (enum tree_code code, /* Don't bother with the calculations inside sizeof; they'll ICE if the source type is incomplete and the pointer value doesn't matter. */ if (skip_evaluation) - return build_nop (build_pointer_type (target_type), expr); + { + expr = build_nop (build_pointer_type (target_type), expr); + if (!want_pointer) + expr = build_indirect_ref (expr, NULL); + return expr; + } /* Do we need to check for a null pointer? */ if (want_pointer && !nonnull) diff --git a/gcc/testsuite/g++.dg/inherit/sizeof1.C b/gcc/testsuite/g++.dg/inherit/sizeof1.C new file mode 100644 index 000000000000..06d5c9993d98 --- /dev/null +++ b/gcc/testsuite/g++.dg/inherit/sizeof1.C @@ -0,0 +1,17 @@ +// PR c++/35007 + +struct AffEntry +{ + union { + char base[256]; + } conds; +}; + +struct PfxEntry +: public AffEntry +{ + PfxEntry() + { + sizeof(conds.base[0]); + } +};