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]

Re: Fold acc_on_device

From: Andrew MacLeod <amacleod at redhat dot com>
To: gcc-patches at gcc dot gnu dot org
Date: Thu, 1 Oct 2015 17:08:14 -0400
Subject: Re: Fold acc_on_device
Authentication-results: sourceware.org; auth=none
References: <560AD6A6 dot 4050109 at acm dot org> <CAFiYyc1vH1nXqtb6C99+rXozyJ5tyV876cLr=NeuB8oC8=3V_g at mail dot gmail dot com> <560BD31C dot 9020503 at acm dot org> <CAFiYyc3XXiyUp9eD3_i2_gNJWY4vJf4p1gKoBZKn7RDCmD=EaA at mail dot gmail dot com> <560D669B dot 6030709 at redhat dot com> <560D692C dot 4090101 at acm dot org>

On 10/01/2015 01:11 PM, Nathan Sidwell wrote:

On 10/01/15 13:00, Andrew MacLeod wrote:
btw, not that it's necessarily important, but I'm about to submit theinclude
reduction patches today,  and it turns out this line is the first use of
anything from cgraph.h in builtins.c.
So if this is "the way" of doing the test, be aware it adds adependency on
cgraph.h that wasn't there before.
The patch I just committed has moved this to gimple-fold.c. Thatappears to already include cgraph.h.

And it does indeed remove the need for cgraph.h in builtins.c.

Andrew

References:
- Re: Fold acc_on_device
  - From: Andrew MacLeod
- Re: Fold acc_on_device
  - From: Nathan Sidwell

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