<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Feb 13, 2014 at 2:15 PM, Robert Lytton <span dir="ltr"><<a href="mailto:robert@xmos.com" target="_blank">robert@xmos.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">




<div>
<div style="direction:ltr;font-size:10pt;font-family:Tahoma">Hi Daniel,<br>
<br>
<div style="font-size:16px;font-family:'Times New Roman'">
<div>
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote"><div class="">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
  ifdef USER_MODE_EMULATION<br>
  RUNSAFELY := $(RUNSAFELY) -t "$(TIMEIT)"<br>
  else<br>
  RUNSAFELY := $(RUNSAFELY) -t "$(TIMEIT_TARGET)"<br>
  endif<br>
</blockquote>
<div>> You forgot to also look at where TIMEIT is used, which includes this line:
</div>
<div>> RUNSAFELYLOCAL := $(RUNSAFELYLOCAL) --show-errors -t "$(TIMEIT)"</div>
<div>> With USER_MODE_EMULATION off, both can be used.</div>
<br></div>
Quite right, my mistake, I originally had:<span dir="ltr"><div class=""><br>
  -all:: timeit timeit-target fpcmp TargetConfig.mk<br>
  +ifdef USER_MODE_EMULATION<br>
  +all:: timeit fpcmp TargetConfig.mk<br>
  +else<br></div>
  +all:: </span><span dir="ltr"><span dir="ltr">timeit </span>timeit-target fpcmp TargetConfig.mk<br>
  +endif<br>
</span><br>
Would this be acceptable?<br></div></div></div></div></div></div></div></blockquote><div><br></div><div>This is ok, but I think it can be more cleanly written as:</div><div>--</div><div> <span style="font-family:'Times New Roman';font-size:16px">all:: timeit fpcmp TargetConfig.mk</span></div>
<div><span style="font-family:'Times New Roman';font-size:16px"><br></span></div><div><font face="Times New Roman" size="3"># Avoid depending on timeit-target if it is not needed, some targets that use emulation cannot build it.</font></div>
<div><font face="Times New Roman" size="3">ifndef USER_MODE_EMULATION</font></div><div><font face="Times New Roman" size="3">all:: timeit-target</font></div><div><font face="Times New Roman" size="3">--</font></div><div><font face="Times New Roman" size="3"><br>
</font></div><span style="font-family:'Times New Roman';font-size:16px">   - Daniel</span></div><div class="gmail_quote"><font face="Times New Roman" size="3"><br></font><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div><div style="direction:ltr;font-size:10pt;font-family:Tahoma"><div style="font-size:16px;font-family:'Times New Roman'"><div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">
Would '<span dir="ltr"><span dir="ltr"> </span>timeit-target' ever be run on an emulator?<br>
I'm trying to understand how remote testing and the use of an emulator would combine...<span class=""><font color="#888888"><br>
</font></span></span><span class=""><font color="#888888"><br>
Robert<br>
<br>
</font></span></div>
</div>
</div>
</div>
</div>
</div>
</div>

</blockquote></div><br></div></div>