[Bug c/20303] Can't push more than 16 nested visibility
--- Additional Comments From jakub at redhat dot com 2005-07-14 15:33 --- Any reason why this hasn't been fixed on gcc-4_0-branch as well? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20303
[Bug c/20303] Can't push more than 16 nested visibility
-- What|Removed |Added Target Milestone|--- |4.1.0 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20303
[Bug c/20303] Can't push more than 16 nested visibility
--- Additional Comments From hjl at lucon dot org 2005-05-30 18:52 --- Fixed on mainline. -- What|Removed |Added Status|NEW |RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20303
[Bug c/20303] Can't push more than 16 nested visibility
--- Additional Comments From hjl at lucon dot org 2005-05-30 18:51 --- Fixed on mainline. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20303
[Bug c/20303] Can't push more than 16 nested visibility
--- Additional Comments From cvs-commit at gcc dot gnu dot org 2005-05-30 18:49 --- Subject: Bug 20303 CVSROOT:/cvs/gcc Module name:gcc Changes by: [EMAIL PROTECTED] 2005-05-30 18:49:04 Modified files: gcc: ChangeLog c-pragma.c gcc/doc: invoke.texi Log message: 2005-05-30 H.J. Lu <[EMAIL PROTECTED]> PR middle-end/20303 * c-pragma.c: Include "vec.h". (handle_pragma_visibility): Use VEC. * doc/invoke.texi: Remove the nested visibility push limit. Patches: http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.8947&r2=2.8948 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/c-pragma.c.diff?cvsroot=gcc&r1=1.85&r2=1.86 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/doc/invoke.texi.diff?cvsroot=gcc&r1=1.626&r2=1.627 -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20303
[Bug c/20303] Can't push more than 16 nested visibility
--- Additional Comments From hjl at lucon dot org 2005-04-05 18:36 --- A new patch is posted at http://gcc.gnu.org/ml/gcc-patches/2005-04/msg00567.html -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20303
[Bug c/20303] Can't push more than 16 nested visibility
--- Additional Comments From giovannibajo at libero dot it 2005-03-03 23:00 --- (In reply to comment #4) I strongly agree with everything in JSM's post. There is simply no reason for keeping such a stupid limit, and even less for manually optimizing conditions to enhance the limit. Using a VEC is more than enough for this case. -- What|Removed |Added Status|UNCONFIRMED |NEW Ever Confirmed||1 Last reconfirmed|-00-00 00:00:00 |2005-03-03 23:00:45 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20303
[Bug c/20303] Can't push more than 16 nested visibility
--- Additional Comments From joseph at codesourcery dot com 2005-03-03 22:58 --- Subject: Re: Can't push more than 16 nested visibility On Thu, 3 Mar 2005, pinskia at gcc dot gnu dot org wrote: > This is a documented behavior. Arbitrary limits are still generally undesirable, even when documented. Some limits are very unlikely to be reached by real programs, even computer-generated ones, and genuinely motivated by efficiency. For example, the C front end only supports 268435455 (= 2^28 - 1) nested scopes. If there were a real use for more scopes, or for a very few programs to exceed other such limits motivated by efficiency concerns, we could add a --enable-huge-programs configure option to increase those limits at some performance cost to almost all normal programs. Another limit in a similar situation is the limit of USHRT_MAX parameters to a preprocessor macro. In this case, the limit is reached by real code and I very much doubt (without profiling information to show to the contrary) that there would be any measurable performance difference with using VEC, so we should just do that and get rid of the limit. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20303
[Bug c/20303] Can't push more than 16 nested visibility
--- Additional Comments From hjl at lucon dot org 2005-03-03 19:19 --- A patch is posted at http://gcc.gnu.org/ml/gcc-patches/2005-03/msg00238.html -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20303
[Bug c/20303] Can't push more than 16 nested visibility
--- Additional Comments From hjl at lucon dot org 2005-03-03 18:47 --- It makes it less useful for header file wrappers. Supporting the case of N (N>16) #pragma GCC visibility push(default) followed by N #pragma GCC visibility pop shouldn't be very hard. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20303
[Bug c/20303] Can't push more than 16 nested visibility
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-03-03 18:43 --- This is a documented behavior. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20303
