On 10/18/13 07:37, Andrew MacLeod wrote:

gimple_check_call_matching_types() was being called from 3 or 4
different files,and seemed more appropriate as a cgraph routine (which
called it 3 times). So I moved that and its helper to cgraph.c.

After that, I only needed to update 4 .c files to directly include
gimple-low.h

bootstraps on x86_64-unknown-linux-gnu with no new regressions. OK?
I'm less sure about this one. I don't see that it clearly belongs in either location. I could easily see it moving out of cgraph.c at some point.

If it furthers your cleanup efforts at this time, that's fine. Just be aware that, at least IMHO, this routine doesn't fit clearly into either location and I wouldn't be surprised if we have to come back to it at some point.

jeff

Reply via email to