public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug middle-end/110379] New: Unnecessary copies after early opts @ 2023-06-23 17:07 hubicka at gcc dot gnu.org 2023-06-23 19:54 ` [Bug middle-end/110379] " pinskia at gcc dot gnu.org ` (2 more replies) 0 siblings, 3 replies; 4+ messages in thread From: hubicka at gcc dot gnu.org @ 2023-06-23 17:07 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110379 Bug ID: 110379 Summary: Unnecessary copies after early opts Product: gcc Version: 13.1.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: middle-end Assignee: unassigned at gcc dot gnu.org Reporter: hubicka at gcc dot gnu.org Target Milestone: --- Following testcase: class a { public: int b; }; void foo (class a *); class b: public a { int test(); }; int b::test() { foo (this); return 0; } produces: int b::test (struct b * const this) { struct a * _1; <bb 2> [local count: 1073741824]: _1 = &this_2(D)->D.2773; foo (_1); return 0; } I think it is equivalent to int b::test (struct b * const this) { <bb 2> [local count: 1073741824]: foo (this); return 0; } Which would save statement and confuse less ipa-sra and ipa-prop. ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug middle-end/110379] Unnecessary copies after early opts 2023-06-23 17:07 [Bug middle-end/110379] New: Unnecessary copies after early opts hubicka at gcc dot gnu.org @ 2023-06-23 19:54 ` pinskia at gcc dot gnu.org 2023-06-23 19:57 ` pinskia at gcc dot gnu.org 2023-06-27 9:40 ` hubicka at gcc dot gnu.org 2 siblings, 0 replies; 4+ messages in thread From: pinskia at gcc dot gnu.org @ 2023-06-23 19:54 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110379 --- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> --- IIRC this is needed for __builtin_object_size still .... ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug middle-end/110379] Unnecessary copies after early opts 2023-06-23 17:07 [Bug middle-end/110379] New: Unnecessary copies after early opts hubicka at gcc dot gnu.org 2023-06-23 19:54 ` [Bug middle-end/110379] " pinskia at gcc dot gnu.org @ 2023-06-23 19:57 ` pinskia at gcc dot gnu.org 2023-06-27 9:40 ` hubicka at gcc dot gnu.org 2 siblings, 0 replies; 4+ messages in thread From: pinskia at gcc dot gnu.org @ 2023-06-23 19:57 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110379 --- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> --- Plus this is basically the same as the following C code: ``` struct a { int t; }; void foo (struct a *); struct b { struct a f; }; void test(struct b *t) { foo(&t->f); } ``` ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug middle-end/110379] Unnecessary copies after early opts 2023-06-23 17:07 [Bug middle-end/110379] New: Unnecessary copies after early opts hubicka at gcc dot gnu.org 2023-06-23 19:54 ` [Bug middle-end/110379] " pinskia at gcc dot gnu.org 2023-06-23 19:57 ` pinskia at gcc dot gnu.org @ 2023-06-27 9:40 ` hubicka at gcc dot gnu.org 2 siblings, 0 replies; 4+ messages in thread From: hubicka at gcc dot gnu.org @ 2023-06-27 9:40 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110379 --- Comment #3 from Jan Hubicka <hubicka at gcc dot gnu.org> --- I thought that ADDR_EXPR of refenence is just fancy way to represent NOP_EXPR or POINTER_PLUS in today gimple. How that affects builtin_object_size? :) However I think ipa-sra will eventually need to handel also ADDR_EXPR that correspnds to non-zero offset ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-06-27 9:40 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-06-23 17:07 [Bug middle-end/110379] New: Unnecessary copies after early opts hubicka at gcc dot gnu.org 2023-06-23 19:54 ` [Bug middle-end/110379] " pinskia at gcc dot gnu.org 2023-06-23 19:57 ` pinskia at gcc dot gnu.org 2023-06-27 9:40 ` hubicka at gcc dot gnu.org
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).