From 12eeb948e48b73988349f35bfc7779feafac9b32 Mon Sep 17 00:00:00 2001 From: Jason Merrill Date: Tue, 29 Jan 2008 00:52:13 -0500 Subject: [PATCH] re PR c++/35007 (Firefox fails to build with affentry.cpp:94: error: ISO C++ forbids subscripting non-lvalue array) PR c++/35007 * class.c (build_base_path): Fix !want_pointer case. From-SVN: r131932 --- gcc/cp/ChangeLog | 5 +++++ gcc/cp/class.c | 7 ++++++- 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index f850ad63bf8a..a93a3d4105e1 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 00b2e3a131b4..ab8c0b84fc01 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) -- 2.47.2