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] [PR8271] Check cv-quals of methods while unifying


On Wed, 7 Jul 2004 05:33:50 +0200, "Giovanni Bajo" <giovannibajo@libero.it> wrote:

> This patch has been tested on i686-pc-linux-gnu with no new regressions. I
> discussed a little with Jason whether we want to patch
> build_method_type_directly to copy the cv-quals within the METHOD_TYPE node.
> This would avoid that ugly chain of macros in my patch (and other parts of the
> frontend) to access them. I tried patching it but there were fallouts in
> cp/mangle.c and similar, so I did not bother at this point.

I still think that's the right way to fix the problem.  As you pointed out,
the member function has const-qualified type, and the trees should reflect
that.

I'm not surprised that there are mangler issues, but that shouldn't be hard
to fix.

> OK for mainline?

No.

Jason


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