public inbox for gcc-prs@sourceware.org help / color / mirror / Atom feed
From: grigory@stl.sarov.ru To: gcc-gnats@gcc.gnu.org Subject: c++/7470: Vtable: virtual function pointers not in declaration order Date: Fri, 02 Aug 2002 02:36:00 -0000 [thread overview] Message-ID: <20020802092753.12015.qmail@sources.redhat.com> (raw) >Number: 7470 >Category: c++ >Synopsis: Vtable: virtual function pointers not in declaration order >Confidential: no >Severity: serious >Priority: medium >Responsible: unassigned >State: open >Class: sw-bug >Submitter-Id: net >Arrival-Date: Fri Aug 02 02:36:01 PDT 2002 >Closed-Date: >Last-Modified: >Originator: Grigory Zagorodnev >Release: 3.1.1 >Organization: >Environment: Red Hat Linux release 7.1 (Seawolf) Kernel 2.4.2-2smp on a 2-processor i686 gcc version 3.1.1 Configured with: ./configure --prefix=/usr/local --program-suffix=-3.1.1 --enable-threads Thread model: posix >Description: The C++ ABI defines order of virtual function pointers in such way: "2.5.2 Virtual Table Components and Order The order of the virtual function pointers in a virtual table is the order of declaration of the corresponding member functions in the class. There is an entry for any virtual function declared in a class, whether it is a new function or overrides a base class function, unless it overrides a function from the primary base, and conversion between their return types does not require an adjustment." *** G++ 3.1.1 compiler violates given rules *** Let's consider the example. --- fail.cpp ---------------------- struct base { short b; virtual int foo() {} }; struct derived: virtual base { int d; virtual int foo() {} virtual int bar() {} }; ----------------------------------- Note that function declaration order is "foo, bar" and 'base' is not the primary base class for 'derived', since it's not nearly empty class. New we check the virtual table for 'derived' class using "g++-3.1.1 -c -fdump-class-hierarchy fail.cpp". The virtual table is: Vtable for derived derived::_ZTV7derived: 9 entries 0 8 4 0 8 &_ZTI7derived 12 derived::bar() 16 derived::foo() 20 0fffffff8 24 0fffffff8 28 &_ZTI7derived 32 derived::_ZTv0_n12_N7derived3fooEv() We see that virtual functions appear in the order "bar, foo" which is wrong. It's expected to have functions in the declaration order as spcified by the ABI. >How-To-Repeat: 1. Compile given testcase g++-3.1.1 -c -fdump-class-hierarchy fail.cpp 2. Look for 'derived' virtual table in the produced fail.cpp.class file >Fix: >Release-Note: >Audit-Trail: >Unformatted:
next reply other threads:[~2002-08-02 9:36 UTC|newest] Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top 2002-08-02 2:36 grigory [this message] 2002-08-02 3:54 nathan 2002-08-02 8:04 nathan 2002-09-13 16:52 nathan
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20020802092753.12015.qmail@sources.redhat.com \ --to=grigory@stl.sarov.ru \ --cc=gcc-gnats@gcc.gnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).