From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23942 invoked by alias); 6 Sep 2007 21:15:41 -0000 Received: (qmail 23934 invoked by uid 22791); 6 Sep 2007 21:15:40 -0000 X-Spam-Check-By: sourceware.org Received: from smtp-out.google.com (HELO smtp-out.google.com) (216.239.45.13) by sourceware.org (qpsmtpd/0.31) with ESMTP; Thu, 06 Sep 2007 21:15:32 +0000 Received: from zps19.corp.google.com (zps19.corp.google.com [172.25.146.19]) by smtp-out.google.com with ESMTP id l86LFP0U024352 for ; Thu, 6 Sep 2007 14:15:25 -0700 Received: from ug-out-1314.google.com (ugeo2.prod.google.com [10.66.166.2]) by zps19.corp.google.com with ESMTP id l86LF9p8027013 for ; Thu, 6 Sep 2007 14:15:23 -0700 Received: by ug-out-1314.google.com with SMTP id o2so240796uge for ; Thu, 06 Sep 2007 14:15:23 -0700 (PDT) Received: by 10.78.204.7 with SMTP id b7mr479364hug.1189113322947; Thu, 06 Sep 2007 14:15:22 -0700 (PDT) Received: by 10.78.40.6 with HTTP; Thu, 6 Sep 2007 14:15:22 -0700 (PDT) Message-ID: Date: Thu, 06 Sep 2007 21:35:00 -0000 From: "Diego Novillo" To: "Zdenek Dvorak" Subject: Re: [patch] Make OMP expansion work on SSA form Cc: gcc-patches@gcc.gnu.org In-Reply-To: <20070830173636.GA26632@kam.mff.cuni.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <20070820150243.GA21008@kam.mff.cuni.cz> <46D57EC6.8090109@google.com> <20070830173636.GA26632@kam.mff.cuni.cz> 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: 2007-09/txt/msg00556.txt.bz2 On 8/30/07, Zdenek Dvorak wrote: > for testing purposes; I guess I can remove the switch, but I would > prefer to keep it at least for some time. "Some time" meaning until you figure out the expansion problems you mentioned in your original mail? The code is not too much, so I guess it won't hurt. But unless you figure this problem out shortly, I'm afraid that it will just rot away. Isn't the whole purpose of the patch to expand OMP while in SSA form? I suppose that the main autoparallelization patch will require this, right? The rest of the patch looks fine. Thanks.