From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23448 invoked by alias); 27 Jul 2011 16:51:51 -0000 Received: (qmail 23430 invoked by uid 22791); 27 Jul 2011 16:51:50 -0000 X-SWARE-Spam-Status: No, hits=-2.9 required=5.0 tests=ALL_TRUSTED,AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from localhost (HELO gcc.gnu.org) (127.0.0.1) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 27 Jul 2011 16:51:36 +0000 From: "sgunderson at bigfoot dot com" To: gcc-bugs@gcc.gnu.org Subject: [Bug tree-optimization/49872] New: Missed optimization: Could coalesce neighboring memsets X-Bugzilla-Reason: CC X-Bugzilla-Type: new X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: tree-optimization X-Bugzilla-Keywords: X-Bugzilla-Severity: enhancement X-Bugzilla-Who: sgunderson at bigfoot dot com X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Changed-Fields: Message-ID: X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 Date: Wed, 27 Jul 2011 16:51:00 -0000 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org X-SW-Source: 2011-07/txt/msg02365.txt.bz2 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49872 Summary: Missed optimization: Could coalesce neighboring memsets Product: gcc Version: 4.7.0 Status: UNCONFIRMED Severity: enhancement Priority: P3 Component: tree-optimization AssignedTo: unassigned@gcc.gnu.org ReportedBy: sgunderson@bigfoot.com Given the following code: #include struct S { int f[1024]; int g[1024]; }; void func(struct S* s) { memset(s->f, 0, sizeof(s->f)); memset(s->g, 0, sizeof(s->g)); } GCC currently generates two memsets. The code with -O2 is a bit hard to read, so I'm just pasting the -Os assembly for clarity: 00000000 : 0: 55 push %ebp 1: 31 c0 xor %eax,%eax 3: 89 e5 mov %esp,%ebp 5: b9 00 04 00 00 mov $0x400,%ecx a: 57 push %edi b: 8b 7d 08 mov 0x8(%ebp),%edi e: f3 ab rep stos %eax,%es:(%edi) 10: 8b 55 08 mov 0x8(%ebp),%edx 13: 66 b9 00 04 mov $0x400,%cx 17: 81 c2 00 10 00 00 add $0x1000,%edx 1d: 89 d7 mov %edx,%edi 1f: f3 ab rep stos %eax,%es:(%edi) 21: 5f pop %edi 22: 5d pop %ebp 23: c3 ret Ideally GCC should also be able to coalesce this together with memsets not written as memset, e.g. s->g[0] = 0;.