public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
From: "qinzhao at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org> To: gcc-bugs@gcc.gnu.org Subject: [Bug c/108896] provide "element_count" attribute to give more context to __builtin_dynamic_object_size() and -fsanitize=bounds Date: Thu, 02 Mar 2023 17:34:37 +0000 [thread overview] Message-ID: <bug-108896-4-ttciH4vpuJ@http.gcc.gnu.org/bugzilla/> (raw) In-Reply-To: <bug-108896-4@http.gcc.gnu.org/bugzilla/> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108896 --- Comment #8 from qinzhao at gcc dot gnu.org --- (In reply to Martin Uecker from comment #7) > An attribute is certainly simpler and should be easy to add. yes. > > I proposed similar extension for C23 and there was some interest, > but I did not have time to follow up. > > https://www.open-std.org/jtc1/sc22/wg14/www/docs/n2660.pdf very interesting proposal! are there any discussions on this proposal? if so, can you point me to them? > > > Sizeof is not a constant expression in ISO C for a VLA and it is not a > constant expression if the struct contains a VLA (GNU extension). So this > is already the case and nothing would need to change. It would also potentially > avoid mistakes when computing the size of such a struct. agreed. However, my understanding is: VLA is only valid inside a function scope. GCC use a special SAVE_EXPR to record its size expression. and evaluated during runtime only once. when this variable length concept is extended to global scope, not sure how to implement the size expression? need some study here. > But the > rules for initialization are not so clear. shall we make this clear? > > I do not think it is a good idea to differentiate between file scope structs > and others. This would be confusing. Yes. agreed. this proposal basically is to extend the VLA concept from function scope to global scope. is my understanding correct? > > Considering that the GNU extensions is rarely used, one could consider > redefining the meaning of > > int n = 1; > struct { > int n; > char buf[n]; > }; > > so that the 'n' refers to the member. Or we add a new syntax similar to > designators (which intuitively makes sense to me). designator might be better IMO. a question here is: for the following nested structure: struct object { ... char items; ... struct inner { ... int flex[]; }; } *ptr; what kind of syntax is good to represent the upper bound of "flex" in the inner struct with "items" in the outer structure? any suggestion?
next prev parent reply other threads:[~2023-03-02 17:34 UTC|newest] Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-02-22 21:26 [Bug c/108896] New: " kees at outflux dot net 2023-02-22 21:31 ` [Bug c/108896] " kees at outflux dot net 2023-02-22 21:32 ` pinskia at gcc dot gnu.org 2023-02-23 8:44 ` rguenth at gcc dot gnu.org 2023-02-23 9:10 ` jakub at gcc dot gnu.org 2023-02-24 15:44 ` muecker at gwdg dot de 2023-03-01 22:54 ` qinzhao at gcc dot gnu.org 2023-03-01 23:27 ` kees at outflux dot net 2023-03-02 15:50 ` muecker at gwdg dot de 2023-03-02 17:34 ` qinzhao at gcc dot gnu.org [this message] 2023-03-02 18:17 ` muecker at gwdg dot de 2023-03-02 18:34 ` muecker at gwdg dot de 2023-03-02 19:47 ` qinzhao at gcc dot gnu.org 2023-03-02 19:56 ` qinzhao at gcc dot gnu.org 2023-03-02 20:07 ` muecker at gwdg dot de 2023-03-03 20:27 ` isanbard at gmail dot com 2023-03-03 21:32 ` muecker at gwdg dot de 2023-03-03 23:18 ` isanbard at gmail dot com 2023-03-04 7:52 ` muecker at gwdg dot de 2023-03-06 19:15 ` isanbard at gmail dot com 2023-03-06 19:18 ` jakub at gcc dot gnu.org 2023-03-06 19:38 ` muecker at gwdg dot de 2023-03-06 19:57 ` muecker at gwdg dot de 2023-03-06 20:05 ` siddhesh at gcc dot gnu.org 2023-03-08 16:56 ` qinzhao at gcc dot gnu.org 2023-03-08 17:13 ` qinzhao at gcc dot gnu.org 2023-03-08 17:36 ` qinzhao at gcc dot gnu.org 2023-03-08 17:38 ` qinzhao at gcc dot gnu.org 2023-03-08 17:43 ` qinzhao at gcc dot gnu.org 2023-03-08 17:48 ` muecker at gwdg dot de 2023-03-08 18:37 ` muecker at gwdg dot de 2023-03-08 19:20 ` qinzhao at gcc dot gnu.org 2023-03-08 19:47 ` qinzhao at gcc dot gnu.org 2023-03-08 20:20 ` muecker at gwdg dot de 2023-03-08 20:47 ` qinzhao at gcc dot gnu.org 2023-03-29 16:12 ` muecker at gwdg dot de 2023-04-03 20:29 ` qinzhao at gcc dot gnu.org 2023-04-03 21:53 ` muecker at gwdg dot de 2023-04-04 15:07 ` qinzhao at gcc dot gnu.org 2023-04-04 16:33 ` muecker at gwdg dot de 2023-04-04 20:08 ` qinzhao at gcc dot gnu.org 2023-04-19 16:32 ` qinzhao at gcc dot gnu.org 2023-05-03 13:57 ` qinzhao at gcc dot gnu.org 2023-05-03 15:32 ` kees at outflux dot net 2023-05-04 15:16 ` muecker at gwdg dot de 2023-05-04 15:30 ` qinzhao at gcc dot gnu.org 2023-05-25 18:14 ` qinzhao at gcc dot gnu.org 2023-05-25 18:47 ` ndesaulniers at google dot com 2023-10-05 19:54 ` tg at mirbsd dot org 2023-10-05 20:21 ` muecker at gwdg dot de 2023-12-27 6:31 ` sean@rogue-research.com 2024-03-06 14:40 ` qinzhao at gcc dot gnu.org
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=bug-108896-4-ttciH4vpuJ@http.gcc.gnu.org/bugzilla/ \ --to=gcc-bugzilla@gcc.gnu.org \ --cc=gcc-bugs@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).