This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

[PATCH, i386]: Fix PR 57106, -fcompare-debug failure with -O2 -fschedule-insns -funroll-all-loops

From: Uros Bizjak <ubizjak at gmail dot com>
To: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Cc: Yuri Rumyantsev <ysrumyan at gmail dot com>, Igor Zamyatin <izamyatin at gmail dot com>
Date: Fri, 3 May 2013 16:00:29 +0200
Subject: [PATCH, i386]: Fix PR 57106, -fcompare-debug failure with -O2 -fschedule-insns -funroll-all-loops

Hello!

Apparently, the wrong insn is used when adding insn dependencies. The
"last" index gets updated to the found !NONDEBUG_INSN. Use "first_arg"
that points to the right insn instead.

2013-05-03  Uros Bizjak  <ubizjak@gmail.com>

    PR target/57106
    * config/i386/i386.c (add_parameter_dependencies): Add dependence
    between "first_arg" and "insn", not "last" and "insn".

testsuite/ChangeLog:

2013-05-03  Uros Bizjak  <ubizjak@gmail.com>

    * gcc.target/i386/pr57106.c: New test.

Tested on x86_64-pc-linux-gnu {,-m32}. I will wait for Intel guys to
check this patch with -fschedule-insns on their codes.

Uros.

Attachment: p.diff.txt
Description: Text document

Follow-Ups:
- Re: [PATCH, i386]: Fix PR 57106, -fcompare-debug failure with -O2 -fschedule-insns -funroll-all-loops
  - From: Yuri Rumyantsev

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]