From patchwork Wed Sep 15 20:46:57 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: reza yazdani X-Patchwork-Id: 64903 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) by ozlabs.org (Postfix) with SMTP id C9D9BB6EFE for ; Thu, 16 Sep 2010 06:47:17 +1000 (EST) Received: (qmail 1204 invoked by alias); 15 Sep 2010 20:47:14 -0000 Received: (qmail 1192 invoked by uid 22791); 15 Sep 2010 20:47:13 -0000 X-SWARE-Spam-Status: No, hits=0.1 required=5.0 tests=AWL, BAYES_05, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, FREEMAIL_FROM, RCVD_IN_DNSWL_NONE, T_RP_MATCHES_RCVD, T_TO_NO_BRKTS_FREEMAIL X-Spam-Check-By: sourceware.org Received: from web33001.mail.mud.yahoo.com (HELO web33001.mail.mud.yahoo.com) (209.191.69.116) by sourceware.org (qpsmtpd/0.43rc1) with SMTP; Wed, 15 Sep 2010 20:47:00 +0000 Received: (qmail 171 invoked by uid 60001); 15 Sep 2010 20:46:58 -0000 Message-ID: <198094.158.qm@web33001.mail.mud.yahoo.com> Received: from [139.95.251.21] by web33001.mail.mud.yahoo.com via HTTP; Wed, 15 Sep 2010 13:46:57 PDT Date: Wed, 15 Sep 2010 13:46:57 -0700 (PDT) From: reza yazdani Subject: Bug bootstrap/45680 To: gcc-patches@gcc.gnu.org Cc: sebastian.pop@amd.com, ChangpengFang MIME-Version: 1.0 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Function min_insn_size is defined under "#ifdef ASM_OUTPUT_MAX_SKIP_PAD" flag. Dispatch Scheduler called the function after #endif. In this fix we move dispatch scheduler code under the same #ifdef. Testing ------- Self compile ran with “–mdispatch-scheduling -fschedule-insns -fsched-pressure –O2". Dispatch scheduling flag was manually set on in the self compile to exercise the new code. No new test added for this implementation. Make check of i386 tests passes. No difference in the number of failures with and without the dispatch flag. ChangeLog --------- 2010-09-15 Reza Yazdani * i386.c (#ifdef ASM_OUTPUT_MAX_SKIP_PAD): Added. Is it okay to commit to trunk? Reza Index: config/i386/i386.c =================================================================== --- config/i386/i386.c (revision 164315) +++ config/i386/i386.c (working copy) @@ -31541,6 +31541,7 @@ ix86_enum_va_list (int idx, const char * return 0; } +#ifdef ASM_OUTPUT_MAX_SKIP_PAD #undef TARGET_SCHED_DISPATCH #define TARGET_SCHED_DISPATCH has_dispatch @@ -32341,7 +32342,7 @@ has_dispatch (rtx insn, int action) return false; } - +#endif /* ??? No autovectorization into MMX or 3DNOW until we can reliably place emms and femms instructions. */