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: [lto] Add timevars and propagate -v and -save-temps


On Sun, Nov 30, 2008 at 18:50, Ollie Wild <aaw@google.com> wrote:

> This only works if MAKEFLAGS is inherited from the parent make.  It
> looks like you broke this mechanism in a previous change and are now
> hacking around that fact.  I'm not quite sure what the issue was that
> prompted unsetting MAKEFLAGS, but maybe a better solution is to unset
> MAKEFLAGS in the test harness.

No, MAKEFLAGS was confusing the Makefile generated by the
ltrans-driver script for an unrelated reason.  It would cause spurious
failures in the GCC testsuite, especially when the parent make was
running with -j N.  Rafael unset it in a previous patch.  My patch
just added some simple parallelism since we couldn't inherit it using
MAKEFLAGS.

Perhaps a better solution would be to determine what exactly was
confusing the Makefile generated by ltrans-driver, but I didn't
explore that.  I'll see about it shortly.


Diego.


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