[Bug c++/35368] [4.1/4.2/4.3/4.4 Regression] With #pragma visibility, `vtable for __cxxabiv1::__class_type_info' is emitted as a hidden-visibility relocation

2008-02-26 Thread benjamin at smedbergs dot us
--- Comment #8 from benjamin at smedbergs dot us 2008-02-26 17:25 --- Yes, to make it clear: the class typeinfo object may have hidden visibility... it's the __cxxabiv1::__class_type_info "class" that should have default visibility always. -- http://gcc.g

[Bug c++/35368] With #pragma visibility, `vtable for __cxxabiv1::__class_type_info' is emitted as a hidden-visibility relocation

2008-02-25 Thread benjamin at smedbergs dot us
--- Comment #2 from benjamin at smedbergs dot us 2008-02-25 15:33 --- Coworker reports the testcase works correctly on gcc4.2.1 also. -- benjamin at smedbergs dot us changed: What|Removed |Added

[Bug c++/35368] With #pragma visibility, `vtable for __cxxabiv1::__class_type_info' is emitted as a hidden-visibility relocation

2008-02-25 Thread benjamin at smedbergs dot us
--- Comment #1 from benjamin at smedbergs dot us 2008-02-25 15:17 --- Created an attachment (id=15229) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15229&action=view) Minimized testcase -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35368

[Bug c++/35368] New: With #pragma visibility, `vtable for __cxxabiv1::__class_type_info' is emitted as a hidden-visibility relocation

2008-02-25 Thread benjamin at smedbergs dot us
relocation Product: gcc Version: 4.3.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c++ AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: benjamin at smedbergs dot us http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35368

[Bug c++/28687] [4.2 regression] dynamic_cast disallowed too rigorously with -fno-rtti

2006-08-23 Thread benjamin at smedbergs dot us
--- Comment #19 from benjamin at smedbergs dot us 2006-08-23 14:23 --- Fixed on trunk for 4.2 -- benjamin at smedbergs dot us changed: What|Removed |Added

[Bug c++/28687] [4.2 regression] dynamic_cast disallowed too rigorously with -fno-rtti

2006-08-11 Thread benjamin at smedbergs dot us
--- Comment #13 from benjamin at smedbergs dot us 2006-08-11 14:21 --- ok, leaving aside the pedantic issue of whether this is a "regression" or not, can we get it fixed? I just posted a patch to gcc-patches which allows dynamic_cast in cases where RTTI is not required, and a

[Bug c++/28687] [4.2 regression] dynamic_cast disallowed too rigorously with -fno-rtti

2006-08-11 Thread benjamin at smedbergs dot us
--- Comment #11 from benjamin at smedbergs dot us 2006-08-11 11:44 --- I'm not claiming that the behavior isn't conformant to the docs, I'm claiming that you regressed a construct that 1) doesn't need RTTI at all (in practice) 2) is used by a major software projec

[Bug c++/28687] [4.2 regression] dynamic_cast disallowed too rigorously with -fno-rtti

2006-08-11 Thread benjamin at smedbergs dot us
--- Comment #8 from benjamin at smedbergs dot us 2006-08-11 10:19 --- The documentation is incorrect. RTTI is not required to find the most-derived class pointer, because the vtable contains this information natively and does not need to know anything about the type. I'm maki

[Bug c++/28687] [4.2 regression] dynamic_cast disallowed too rigorously with -fno-rtti

2006-08-10 Thread benjamin at smedbergs dot us
--- Comment #6 from benjamin at smedbergs dot us 2006-08-11 01:50 --- Gabriel, can you explain how it worked correctly in all the previous releases? Is my summary of how it is done using only the vtable inaccurate? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28687

[Bug c++/28687] [4.2 regression] dynamic_cast disallowed too rigorously with -fno-rtti

2006-08-10 Thread benjamin at smedbergs dot us
--- Comment #3 from benjamin at smedbergs dot us 2006-08-10 20:46 --- This cast in particular doesn't need RTTI, because the offset to the most derived class is in the vtable. Mozilla uses this cast to determine the concrete type for logging purposes, without needing or wanting

[Bug c++/28687] [4.2.0 regression] dynamic_cast disallowed too rigorously with -fno-rtti

2006-08-10 Thread benjamin at smedbergs dot us
--- Comment #1 from benjamin at smedbergs dot us 2006-08-10 20:22 --- Created an attachment (id=12059) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=12059&action=view) preprocessed testcase -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28687

[Bug c++/28687] New: [4.2.0 regression] dynamic_cast disallowed too rigorously with -fno-rtti

2006-08-10 Thread benjamin at smedbergs dot us
rigorously with -fno-rtti Product: gcc Version: 4.2.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c++ AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: benjamin at smedbergs dot us http

[Bug c++/26905] default-visibility class symbol improperly resolved as hidden-visibility

2006-07-04 Thread benjamin at smedbergs dot us
--- Comment #5 from benjamin at smedbergs dot us 2006-07-05 02:49 --- Jason, is this a patch that can be safely considered for the 1.4.1/1.4.0 branches? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26905

[Bug middle-end/26905] default-visibility class symbol improperly resolved as hidden-visibility

2006-03-28 Thread benjamin at smedbergs dot us
--- Comment #1 from benjamin at smedbergs dot us 2006-03-28 16:09 --- Created an attachment (id=11138) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=11138&action=view) Reduced testcase, .i gcc -fPIC -shared -o libfoo.so nsAppRunner-test6.i Tested with 4.1 branch after the

[Bug middle-end/26905] New: default-visibility class symbol improperly resolved as hidden-visibility

2006-03-28 Thread benjamin at smedbergs dot us
hidden-visibility Product: gcc Version: 4.1.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component: middle-end AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: benjamin at smedbergs dot us

[Bug middle-end/20297] #pragma GCC visibility isn't properly handled for builtin functions

2005-09-13 Thread benjamin at smedbergs dot us
-- What|Removed |Added Attachment #9720 is|1 |0 patch|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20297

[Bug middle-end/20297] #pragma GCC visibility isn't properly handled for builtin functions

2005-09-13 Thread benjamin at smedbergs dot us
--- Additional Comments From benjamin at smedbergs dot us 2005-09-13 16:34 --- Created an attachment (id=9720) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=9720&action=view) Failure even with the latest patch I'm wrong. attachment 9035 fixes the compile errors

[Bug middle-end/20297] #pragma GCC visibility isn't properly handled for builtin functions

2005-09-09 Thread benjamin at smedbergs dot us
--- Additional Comments From benjamin at smedbergs dot us 2005-09-09 20:51 --- I can confirm that attachment 9035 fixes the mozilla compile issues with GCC 4.0/4.0.1/trunk. -- What|Removed |Added