This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [C++ PATCH] Fix 21799

From: "Giovanni Bajo" <rasky at develer dot com>
To: "Nathan Sidwell" <nathan at codesourcery dot com>
Cc: <gcc-patches at gcc dot gnu dot org>
Date: Fri, 8 Jul 2005 15:34:38 +0200
Subject: Re: [C++ PATCH] Fix 21799
References: <42CE5E8A.6050406@codesourcery.com> <018a01c583be$aaec3980$bf03030a@trilan> <42CE7F3E.60907@codesourcery.com>

Nathan Sidwell <nathan@codesourcery.com> wrote:

>> Well you're special casing method_type. The correct fix would be to
encode
>> the CV qualification within the METHOD_TYPE instead of within the "this"
>> pointer in the arguments. If we did this, we would not need any special
>> case.
>
> true, but that'd be a bigger change I wouldn't want to push into 4.0.  I'm
> not immediately sure how not having the cvquals on the arguments will
affect
> argument passing promotions.

In fact, I'd have cvquals *both* on the METHOD_TYPE *and* the this pointer.
Sorry if my previous mail was confusing in this regard. I'd expect this
change to be relatively trivial (probably in the same order of the patch you
just committed).
-- 
Giovanni Bajo

References:
- [C++ PATCH] Fix 21799
  - From: Nathan Sidwell
- Re: [C++ PATCH] Fix 21799
  - From: Giovanni Bajo
- Re: [C++ PATCH] Fix 21799
  - From: Nathan Sidwell

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]