Hi Alastair,<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 30, 2012 at 1:07 PM, Alastair Murray <span dir="ltr"><<a href="mailto:alastairmurray42@gmail.com" target="_blank">alastairmurray42@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Manman, all,<br>
<br>
Daniel: I have also CC'ed you on this as you seem to do most of the test-suite work.  If you have no interest in these Makefiles or you feel they are not suitable for the repository then please let me know.</blockquote>
<div><br></div><div>The test-suite portion looks fine to me. From my perspective, anyone is free to add new TEST= flavors to the test-suite if they think they might be useful to other people.</div><div><br></div><div>I defer to Manman's comments on whether or not the patches actually work. Note that when she is referring to SPEC, she means as integrated into the test-suite. The SPEC portion has some particularities so just because something works for the rest of the test-suite doesn't guarantee it will work with SPEC. However, I didn't actually try to run with these patches so I can't comment on any particular issues.</div>
<div><br></div><div> - Daniel</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
<br>
On 07/09/12 20:54, Manman Ren wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Alastair, I looked at your test-suite patch for PGO, it needs to be modified to make it work for SPEC (Makefile.spec).<br>
The modifications are attached.<br>
</blockquote>
<br></div>
Manman: I have included your SPEC changes in the patch.  I don't have access to SPEC so I can't test them, but I have visually verified that your previous patch is still compatible with recent test-suite changes.<div class="im">
<br>
<br>
<br>
<br>
> I haven't found time to make it work for both SPEC and other benchmarks.<br>
<br></div>
Manman, I don't understand this statement.  The patch as attached works for test-suite, I assume that you have tested it on SPEC.  So it works for both ...<br>
<br>
Otherwise the only change from the previous patch is to switch uses of $(RUNTOOLSAFELY) over to $(RUNSAFELYLOCAL) to match recent test-suite changes.<br>
<br>
This patch is still dependent on the as-yet unreviewed "[PATCH 3/4] Profile metadata: runtime support" patch.  It could be trivially rewritten to not depend on that patch, but make -jN would not work for N>1.<br>

<br>
Regards,<br>
Alastair.<br>
<br>
</blockquote></div><br></div>