From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21418 invoked by alias); 21 Mar 2012 00:14:14 -0000 Received: (qmail 21408 invoked by uid 22791); 21 Mar 2012 00:14:13 -0000 X-SWARE-Spam-Status: No, hits=-2.0 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,RCVD_IN_DNSWL_LOW,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mail-iy0-f175.google.com (HELO mail-iy0-f175.google.com) (209.85.210.175) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 21 Mar 2012 00:13:58 +0000 Received: by iaag37 with SMTP id g37so792816iaa.20 for ; Tue, 20 Mar 2012 17:13:57 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:x-system-of-record:x-gm-message-state; bh=zeVP7CwUaB37Oxlnb4X5q8EjOLdpJLEr56EIZdxgLwM=; b=pEg3HB+zSKmcnXEGPYwyp894XzfFHBe9VPDRmDq/eJFAqD99m2YVQWs2YJmnAn7dRr M4d1bcjlPXYheOECCkCAZZGg7nas4KsG3yfCJEnrutDimiRj4OQA3ivqMIqdpoOSQWBV yGnI2bGRu5Kn8ziWUBcBPs+iT4D/RiI1lZoGK/pk10R8KZXZ1BtYZU1Xvnc3DAs/bbws /PFzUytr9n85vLkqYI8xJRf83VgBppQKSVOD7S2Z3v8idHSNhNsxFENMedW82GMsn7qK 1ZF5rlbt6JrpNqQqr2qxyl5/sMZXDezo4qF5HEWnd2znG1rJMBMqs3IKJ5MLQlAOa0NS qN8w== Received: by 10.50.170.97 with SMTP id al1mr1378566igc.33.1332288837777; Tue, 20 Mar 2012 17:13:57 -0700 (PDT) MIME-Version: 1.0 Received: by 10.50.170.97 with SMTP id al1mr1378560igc.33.1332288837711; Tue, 20 Mar 2012 17:13:57 -0700 (PDT) Received: by 10.231.178.103 with HTTP; Tue, 20 Mar 2012 17:13:57 -0700 (PDT) In-Reply-To: References: Date: Wed, 21 Mar 2012 00:14:00 -0000 Message-ID: Subject: Re: [patch] Split parts of cse_insn out to a few new functions From: Ian Lance Taylor To: Steven Bosscher Cc: GCC Patches Content-Type: text/plain; charset=ISO-8859-1 X-System-Of-Record: true X-Gm-Message-State: ALoCoQmJQOF+8CHmkkwqXoyFYyHg1v1fDCdQPzY1h0NkAmsWizNob/efySfRXN1OI0H/O9nlexchesmqCKDnmAw8bhLoq7PGU83TehPHGxZWIyHxIwS4u32WHX0MRcuf9FUB8G49lBH3GmgAeseuq4BrEWHCFgAUVw== X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2012-03/txt/msg01401.txt.bz2 On Tue, Mar 20, 2012 at 2:06 PM, Steven Bosscher wrote: > > This patch splits a couple of pieces of cse_insn out to new functions. > There are no functional changes, and no code generation differences as > far as I could tell on x86_64 (-m64 and -m32). > > The purpose of the patch is and, loto hopefully make cse_insn easier > to understand. In a follow-up patch, I will make canonicalize_insn run > only once per insn (it currently, i.e. before and after this patch, > runs multiple times for CSE on extended basic blocks if a block is in > multiple extended basic blocks). This is OK. Thanks. Ian