From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1069 invoked by alias); 29 Nov 2013 19:56:37 -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 Received: (qmail 1031 invoked by uid 48); 29 Nov 2013 19:56:34 -0000 From: "ubizjak at gmail dot com" To: gcc-bugs@gcc.gnu.org Subject: [Bug target/59163] [4.8/4.9 Regression] program compiled with g++ -O3 segfaults Date: Fri, 29 Nov 2013 19:56:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: target X-Bugzilla-Version: 4.8.2 X-Bugzilla-Keywords: wrong-code X-Bugzilla-Severity: normal X-Bugzilla-Who: ubizjak at gmail dot com X-Bugzilla-Status: ASSIGNED X-Bugzilla-Priority: P2 X-Bugzilla-Assigned-To: jakub at gcc dot gnu.org X-Bugzilla-Target-Milestone: 4.8.3 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-SW-Source: 2013-11/txt/msg03066.txt.bz2 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D59163 --- Comment #14 from Uro=C5=A1 Bizjak --- (In reply to Uro=C5=A1 Bizjak from comment #13) > (In reply to Jakub Jelinek from comment #12) > > Created attachment 31332 [details] > > gcc49-pr59163.patch > >=20 > > So like this? >=20 > Yes, with adjusted comment in ix86_legitimate_combined_insn. >=20 > IIRC, unaligned moves won't be propagated during or after reload, so it > looks to me that the approach is correct. Running the testsuite with your patch applied exposed a minor problem: FAIL: gcc.target/i386/sse-1.c scan-assembler-not movaps movlps/movhps and movlpd/movhpd can also handle unaligned operands (please = see ix86_expand_vector_move_misalign). We should simply tag instructions that operate on unaligned operands (attribute type =3D ssemovu) and check type attribute instead. The proposed approach would mean to change all scheduler and attribute calculation checks from "ssemov" to "ssemov,ssemovu", but this would be a simple mechanical change. >>From gcc-bugs-return-436290-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Fri Nov 29 20:06:13 2013 Return-Path: Delivered-To: listarch-gcc-bugs@gcc.gnu.org Received: (qmail 8990 invoked by alias); 29 Nov 2013 20:06:13 -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 Delivered-To: mailing list gcc-bugs@gcc.gnu.org Received: (qmail 8931 invoked by uid 48); 29 Nov 2013 20:06:10 -0000 From: "mpolacek at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug sanitizer/59333] ICE with long long and -m32 -fsanitize=undefined Date: Fri, 29 Nov 2013 20:06:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: sanitizer X-Bugzilla-Version: 4.9.0 X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: mpolacek at gcc dot gnu.org X-Bugzilla-Status: ASSIGNED X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: mpolacek at gcc dot gnu.org X-Bugzilla-Target-Milestone: 4.9.0 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-SW-Source: 2013-11/txt/msg03067.txt.bz2 Content-length: 203 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59333 --- Comment #1 from Marek Polacek --- I think we just shouldn't try to pass values by reference in ubsan_encode_value...