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 the include
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 a dependency on
cgraph.h that wasn't there before.

The patch I just committed has moved this to gimple-fold.c. That appears to already include cgraph.h.


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

Andrew

Reply via email to