<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jan 24, 2014 at 9:31 AM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  It seems the xcore cmath.h file is buggy.<br>
  We are exposing the math.h macro.<br>
  Looking at an upstream version of the math.h header, the log2 macro has an __cplusplus guard around it.<br>
  This is the correct fix.<br></blockquote><div><br></div><div>Great - I assume the same is true for the "infinity" code you have out for review too? You can cancel both these CRs, then?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
  Patch is wrong.<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2618" target="_blank">http://llvm-reviews.chandlerc.com/D2618</a><br>
</blockquote></div><br></div></div>